◐ Shell
reader mode source ↗
Skip to content

bpo-32769: Write annotation entry for glossary#6657

Merged
ilevkivskyi merged 8 commits into
python:masterfrom
andresdelfino:annotations-glosary-entry
May 14, 2018
Merged

bpo-32769: Write annotation entry for glossary#6657
ilevkivskyi merged 8 commits into
python:masterfrom
andresdelfino:annotations-glosary-entry

Conversation

@andresdelfino

@andresdelfino andresdelfino commented Apr 30, 2018

Copy link
Copy Markdown
Contributor

@ilevkivskyi ilevkivskyi left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hide comment

Thanks, looks good! I have just few minor comments.

@andresdelfino

Copy link
Copy Markdown
Contributor Author

@ilevkivskyi should I keep "class attributes" to refer to instance variables, or should I use the later to separate between data attributes (instance variables) and behavior attributes (methods)?

I think "class attributes" is somewhat ambiguous.

@ilevkivskyi ilevkivskyi left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hide comment

I like how this looks now. Thanks!

@ilevkivskyi ilevkivskyi merged commit f2290fb into python:master May 14, 2018
@andresdelfino andresdelfino deleted the annotations-glosary-entry branch May 14, 2018 19:06
andresdelfino added a commit to andresdelfino/cpython that referenced this pull request May 14, 2018
https://bugs.python.org/issue32769
(cherry picked from commit f2290fb)

Co-authored-by: Andrés Delfino <adelfino@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants