◐ Shell
reader mode source ↗
Skip to content

bpo-18374: fix tests to check the correct thing about line numbers#14659

Merged
ilevkivskyi merged 1 commit into
python:masterfrom
cfbolz:bpo-18374-fix-col-offset-binop-fix-tests
Jul 9, 2019
Merged

bpo-18374: fix tests to check the correct thing about line numbers#14659
ilevkivskyi merged 1 commit into
python:masterfrom
cfbolz:bpo-18374-fix-col-offset-binop-fix-tests

Conversation

@cfbolz

@cfbolz cfbolz commented Jul 9, 2019

Copy link
Copy Markdown
Contributor

@ilevkivskyi here's the promised fix.

Should I also try to do the backport to 3.7? (the automated merging fails because the end_lineno and end_col_offset didn't exist in 3.7).

https://bugs.python.org/issue18374

@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!

@miss-islington

Copy link
Copy Markdown
Contributor

Thanks @cfbolz for the PR, and @ilevkivskyi for merging it 🌮🎉.. I'm working now to backport this PR to: 3.8.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jul 9, 2019
…ythonGH-14659)

(cherry picked from commit 430a9f4)

Co-authored-by: Carl Friedrich Bolz-Tereick <cfbolz@gmx.de>
@bedevere-bot

Copy link
Copy Markdown

GH-14672 is a backport of this pull request to the 3.8 branch.

@ilevkivskyi

Copy link
Copy Markdown
Member

@cfbolz

Should I also try to do the backport to 3.7?

I don't think it is worth spending some manual work on the backport. Fixing it in 3.8 is already good enough.

ilevkivskyi pushed a commit that referenced this pull request Jul 9, 2019
…H-14659) (GH-14672)

(cherry picked from commit 430a9f4)

Co-authored-by: Carl Friedrich Bolz-Tereick <cfbolz@gmx.de>
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.

5 participants