Thanks to visit codestin.com
Credit goes to github.com

Skip to content

gh-92032: Add soft keywords to rlcompleter #92029

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

Merged
merged 4 commits into from
May 2, 2022
Merged

gh-92032: Add soft keywords to rlcompleter #92029

merged 4 commits into from
May 2, 2022

Conversation

kbeldan
Copy link
Contributor

@kbeldan kbeldan commented Apr 28, 2022

@ghost
Copy link

ghost commented Apr 28, 2022

All commit authors signed the Contributor License Agreement.
CLA signed

@bedevere-bot
Copy link

Every change to Python requires a NEWS entry.

Please, add it using the blurb_it Web app or the blurb command-line tool.

@kbeldan
Copy link
Contributor Author

kbeldan commented Apr 28, 2022

Hi, I figured this one would skip issue / news

@AlexWaygood
Copy link
Member

I've added "skip issue", but if this is a bugfix that will result in a change in behaviour, I think a NEWS entry would probably be warranted :)

@kbeldan
Copy link
Contributor Author

kbeldan commented Apr 28, 2022

Not a bugfix, with this the interpreter can autocomplete the match / case / _(wildcard pattern), which are as-of-now the only soft keywords added with PEP-634

@AlexWaygood
Copy link
Member

Not a bugfix, with this the interpreter can autocomplete the match / case / _(wildcard pattern), which are as-of-now the only soft keywords added with PEP-634

In which case it is a feature, and there's even more need for it to have a news entry :)

We generally only skip news for things like docs changes, minor typo fixes in comments, changes to the CI setup, etc.. Pretty much everything else needs an entry.

@kbeldan
Copy link
Contributor Author

kbeldan commented Apr 28, 2022

To answer your request, given your repository rules, that would require a GH issue.

@bedevere-bot
Copy link

Every change to Python requires a NEWS entry.

Please, add it using the blurb_it Web app or the blurb command-line tool.

@kbeldan
Copy link
Contributor Author

kbeldan commented Apr 28, 2022

squashed and reworded after NEWS entry generated by blurb-it

@kbeldan kbeldan changed the title Add soft keywords to rlcompleter gh-92032: Add soft keywords to rlcompleter Apr 29, 2022
Let the interpreter autocomplete soft-keywords, ATM the PEP-634 'match'
/ 'case' / '_' (wildcard pattern).
@kbeldan
Copy link
Contributor Author

kbeldan commented May 2, 2022

I have rebased because upstream has evolved since, however I don't know how that affects the process.
At this stage people let their PRs lying in the pile ?

Copy link
Member

@JelleZijlstra JelleZijlstra left a comment

Choose a reason for hiding this comment

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

Tried it out locally and it works, thank you!

@JelleZijlstra
Copy link
Member

I have rebased because upstream has evolved since, however I don't know how that affects the process. At this stage people let their PRs lying in the pile ?

Rebasing is generally discouraged because it requires force pushing, which can make reviews harder. You can click the "update branch" button to bring the branch up to date with main.

@JelleZijlstra JelleZijlstra self-assigned this May 2, 2022
@kbeldan
Copy link
Contributor Author

kbeldan commented May 2, 2022

Last test is stuck
stuck_in_concurrent_futures

@JelleZijlstra
Copy link
Member

Last test is stuck stuck_in_concurrent_futures

I turned it off and on again

@JelleZijlstra JelleZijlstra merged commit 4bed9c4 into python:main May 2, 2022
@AlexWaygood
Copy link
Member

Congrats on your first CPython contribution, @kbeldan! 😃

@AlexWaygood
Copy link
Member

AlexWaygood commented May 3, 2022

Last test is stuck stuck_in_concurrent_futures

I've opened #92222, since this is also happening on my completely unrelated PR here: https://github.com/python/cpython/runs/6270908903?check_suite_focus=true

@JelleZijlstra, do you know if there's any way of getting the logs from the run where the test failed, prior to it being switched off and on again?

@kbeldan kbeldan deleted the rlcomplete_softkws branch May 3, 2022 16:13
@kbeldan kbeldan restored the rlcomplete_softkws branch May 3, 2022 16:19
@kbeldan kbeldan deleted the rlcomplete_softkws branch May 3, 2022 16:19
@kbeldan kbeldan restored the rlcomplete_softkws branch May 3, 2022 16:19
@kbeldan kbeldan deleted the rlcomplete_softkws branch May 3, 2022 16:19
@kbeldan kbeldan restored the rlcomplete_softkws branch May 3, 2022 16:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants