-
-
Notifications
You must be signed in to change notification settings - Fork 107
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
Fixes #103 #104
Conversation
…same as the current github python template)
I've made all the requested changes except using the full qualified name as the object name by default. |
There was a problem hiding this 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.
Thank's a lot! That was smooth! :) |
…ecorator class, refs #107
Sorry, i shouldn't have pushed my changes. It was stupid. Should i reverse my commit to make this PR pass first? |
Yes, please. |
There was a problem hiding this 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.
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. |
Object
instance registration indefinitions
..gitignore
.Pipfile
to make installation testing easier for pipenv users.