Skip to content
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

bpo-39235: Check end_lineno and end_col_offset of AST nodes. #17926

Merged
merged 3 commits into from Jan 10, 2020

Conversation

@serhiy-storchaka
Copy link
Member

serhiy-storchaka commented Jan 9, 2020

@serhiy-storchaka serhiy-storchaka requested a review from gvanrossum Jan 9, 2020
@serhiy-storchaka serhiy-storchaka changed the title bpo-39235 Check end_lineno and end_col_offset of AST nodes. bpo-39235: Check end_lineno and end_col_offset of AST nodes. Jan 9, 2020
@gvanrossum

This comment has been minimized.

Copy link
Member

gvanrossum commented Jan 9, 2020

Should this be backported to 3.8?

Which should be merged first, this one or GH-17925?

Should I still add the test that @pablogsal suggested in GH-17925?

@serhiy-storchaka

This comment has been minimized.

Copy link
Member Author

serhiy-storchaka commented Jan 9, 2020

I do not want to backport this, because I am not sure that all end positions are correct. If there are incorrect end positions, this can break tests for alternate Python implementations which calculate them correctly.

bab2c12 shows the effect of #17925.

@gvanrossum

This comment has been minimized.

Copy link
Member

gvanrossum commented Jan 9, 2020

OK. I suppose you have to recalculate now that GH-17925 has landed?

@serhiy-storchaka

This comment has been minimized.

Copy link
Member Author

serhiy-storchaka commented Jan 10, 2020

Yes, see bab2c12.

@serhiy-storchaka serhiy-storchaka merged commit 850a885 into python:master Jan 10, 2020
8 checks passed
8 checks passed
Windows (x86)
Details
Windows (x64)
Details
macOS
Details
Ubuntu
Details
Azure Pipelines PR #20200109.33 succeeded
Details
bedevere/issue-number Issue number 39235 found
Details
bedevere/news "skip news" label found
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@serhiy-storchaka serhiy-storchaka deleted the serhiy-storchaka:test-ast-end branch Jan 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.