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
A property to access the assets
field of release (#1898)
#1899
Conversation
... in order to avoid extra requests. <PyGithub#1898>
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.
This is looking like a good start, I have one niggle inline, and this requires a test -- if the existing replay data used in tests/GitRelease.py already has assets, then you can assert that and you're golden.
@property | ||
def assets(self): | ||
""" | ||
Already returned assets info (no additional requests will be done). |
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.
Just the :type: here, 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.
Where to put a note on the difference between get_assets()
and assets
though?
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.
Hmm... Hmm...
Just the :type: here, please
How then it should be mentioned which methods do additional requests and which not?
Sorry to remind but GitHub has limits...
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
... in order to avoid extra requests.
#1898