Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixes #103 #104

Closed
wants to merge 8 commits into from
Closed

Fixes #103 #104

wants to merge 8 commits into from

Conversation

volfpeter
Copy link
Contributor

  • Fixed Object instance registration in definitions.
  • Added a couple of entries to .gitignore.
  • Committed a Pipfile to make installation testing easier for pipenv users.

.gitignore Show resolved Hide resolved
Pipfile Outdated Show resolved Hide resolved
sanic_openapi/doc.py Show resolved Hide resolved
sanic_openapi/swagger.py Outdated Show resolved Hide resolved
@volfpeter
Copy link
Contributor Author

I've made all the requested changes except using the full qualified name as the object name by default.

chenjr0719
chenjr0719 previously approved these changes Jun 9, 2019
Copy link
Member

@chenjr0719 chenjr0719 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@volfpeter Thanks, for your contributions. @ahopkins I think this PR is ready to go.

@volfpeter
Copy link
Contributor Author

Thank's a lot! That was smooth! :)

@volfpeter
Copy link
Contributor Author

Sorry, i shouldn't have pushed my changes. It was stupid. Should i reverse my commit to make this PR pass first?

@chenjr0719
Copy link
Member

Yes, please.

Copy link
Member

@ahopkins ahopkins left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would suggest also a git rebase -i or other method to squash the commits.

@volfpeter
Copy link
Contributor Author

I've submitted another PR (#110) instead where I've fixed my own git history and resolved merge conflicts as well. Please close this PR and merge that one instead. Sorry for the issue.

@ahopkins ahopkins closed this Jun 11, 2019
ahopkins added a commit that referenced this pull request Jun 19, 2019
Fixes #103 and the git history mess up of #104
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants