-
Notifications
You must be signed in to change notification settings - Fork 386
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
Remove deprecated Python settings #625
Remove deprecated Python settings #625
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for taking the time to contribute this PR, we appreciate it! ✨
Took the liberty to remove the linters and formatters from DEFAULT_UTILS, but happy to revert if it not appropriate! 😊 |
@samruddhikhandale friendly ping 😊 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the ping, apologies to have missed this!
Left some comments.
This reverts commit 57c7bec.
Last commit reverted 😊 |
@samruddhikhandale friendly ping! |
@luabud I had been waiting for this comment https://github.com/devcontainers/features/pull/625/files#r1286251867 Can you help resolve it, then we could merge this PR. Thanks! |
@samruddhikhandale oh I'm so sorry! I had totally misunderstood that this would be done in a follow up clean up PR (which I now realize was about bumping the major version, instead of minor 🤦♀️) Just bumped the version to it to 1.2.1 :) thanks a lot! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great, thank you so much! 🎉
Hi there!
The
python.linting
andpython.formatting
settings have been deprecated as in the latest Python extension pre-release version, as we're migrating our tooling support to extensions (see here for more details).We figured we'd submit PRs to remove them from devcontainers configs 😊