-
-
Notifications
You must be signed in to change notification settings - Fork 21.4k
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
Add missing descriptions for Image's documentation #86997
Conversation
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.
LGTM
Maybe you can take a look too @BlueCube3310
b3923a3
to
4b82dba
Compare
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.
Great work, just 3 small things:
4b82dba
to
e01b917
Compare
Pushed changes to address the above feedback. Thank you both for providing suggestions! |
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.
Looks great!
Thanks! |
Cherry-picked for 4.2.2. |
Cherry-picked for 4.1.4. |
This PR adds missing descriptions to Image's methods and constants, filling them in.
The only ones I have doubts about are FORMAT_ETC2_RA_AS_RG and FORMAT_DXT5_RA_AS_RG, and code is not exactly clear unless you're savvy enough.
...
Uhm... Yeah, that's all.