bpo-25324: Move the description of tokenize tokens to token.rst. by serhiy-storchaka · Pull Request #1911 · python/cpython
Navigation Menu
{{ message }}
- Notifications You must be signed in to change notification settings
- Fork 34.4k
Merged
serhiy-storchaka merged 2 commits intopython:masterfrom
Jun 6, 2017Merged
bpo-25324: Move the description of tokenize tokens to token.rst.#1911
serhiy-storchaka merged 2 commits intopython:masterfrom
bpo-25324: Move the description of tokenize tokens to token.rst.#1911
serhiy-storchaka merged 2 commits intopython:masterfrom
Conversation
Copy link Copy Markdown
mention-bot
commented
Jun 1, 2017
mention-bot commented
Jun 1, 2017@serhiy-storchaka, thanks for your PR! By analyzing the history of the files in this pull request, we identified @birkenfeld, @benjaminp and @vadmium to be potential reviewers.
the-knights-who-say-ni
added
the
CLA signed
label
vadmium reviewed Jun 3, 2017
Doc/library/token.rst Outdated
| Added :data:`COMMENT`, :data:`NL` and :data:`ENCODING` to bring | ||
| the tokens in the C code in line with the tokens needed in | ||
| :mod:`tokenize` module. These tokens aren't used by the C tokenizer. No newline at end of file | ||
| Following token type values aren't used by the C tokenizer but are needed for |
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.
Add the in front: The following token type values . . .
serhiy-storchaka
merged commit
5cefb6c
into
python:master
serhiy-storchaka
deleted the
token-docs
branch
Copy link Copy Markdown
Member Author
serhiy-storchaka
commented
Jun 6, 2017
serhiy-storchaka commented
Jun 6, 2017Thank you for your review @vadmium.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment