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

Provide a setting to always show the selected interpreter #19089

Closed
brettcannon opened this issue May 9, 2022 · 5 comments · Fixed by #19513
Closed

Provide a setting to always show the selected interpreter #19089

brettcannon opened this issue May 9, 2022 · 5 comments · Fixed by #19513
Assignees
Labels
area-environments Features relating to handling interpreter environments community ask Feature request that the community expressed interest in feature-request Request for new features or functionality verification-needed Verification of issue is requested verified Verification succeeded
Milestone

Comments

@brettcannon
Copy link
Member

As of now the selected interpreter is hidden when you are not viewing Python files or special Python-specific files. But some folks want the interpreter info to show more often, e.g. extensions supporting DSLs requiring Python, Django devs editing their HTML file templates, etc.

So we want to provide a setting (name TBD) that will let folks opt into having the selected interpreter to always be displayed when the Python extension is loaded.

@brettcannon brettcannon added feature-request Request for new features or functionality needs proposal Need to make some design decisions area-environments Features relating to handling interpreter environments labels May 9, 2022
@karrtikr karrtikr added the needs community feedback Awaiting community feedback label May 9, 2022
@wouterdb
Copy link

@brettcannon is there a way we can make this move forward?

@brettcannon
Copy link
Member Author

@wouterdb unfortunately the hold up is entirely on me and @luabud to make a decision about the language status menu and whether/how we want to use it. Once that's decided on then that will inform whether this setting is even necessary. We are planning to make the decision this week or next.

@karrtikr
Copy link

Related: #18930

@brettcannon brettcannon added community ask Feature request that the community expressed interest in and removed needs community feedback Awaiting community feedback labels Jul 14, 2022
@brettcannon
Copy link
Member Author

We've decided to implement this, so votes are not required to make it into our queue, but further votes will help us prioritize getting to it. 🙂

@karrtikr
Copy link

karrtikr commented Jul 25, 2022

Verification steps:

This affects the interpreter information in status bar:
image

  • Install the pre-release version of the extension:
    image
  • Setting name: python.interpreter.infoVisibility.
  • Verify setting is only available at User/Machine level.
  • Verify the description of the setting and each value aligns with the behavior.
  • When set to onPythonRelated, verify information in status bar disappears if non-Python files are in focus (settings.json is considered a Python related file).
  • Verify we don't need to reload for setting change to take effect.

@karrtikr karrtikr added needs PR verification-needed Verification of issue is requested and removed needs proposal Need to make some design decisions labels Jul 25, 2022
@karrtikr karrtikr self-assigned this Jul 25, 2022
@karrtikr karrtikr added this to the July 2022 milestone Jul 25, 2022
@rzhao271 rzhao271 added the verified Verification succeeded label Jul 27, 2022
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 27, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-environments Features relating to handling interpreter environments community ask Feature request that the community expressed interest in feature-request Request for new features or functionality verification-needed Verification of issue is requested verified Verification succeeded
Projects
None yet
4 participants