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

add initialize_conda and register_python keys to control whether to show these options #507

Merged
merged 10 commits into from Aug 30, 2022

Conversation

jaimergp
Copy link
Contributor

All installers will offer the user to initialize conda (all OS) and register Python as default (Windows only). Some applications might be using conda behind the scenes but don't want to expose this to the user, so asking for that kind of options can be strange and surprising if they have never heard about conda before.

This PR adds two keys to prevent this behavior:

  • initialize_conda (True by default). If false, conda init prompts won't appear on sh installers. EXEs and PKGs won't show the checkboxes.
  • register_python (True by default). If false, EXEs won't offer this option, which was discouraged anyway.

@anaconda-issue-bot anaconda-issue-bot added the cla-signed [bot] added once the contributor has signed the CLA label Mar 14, 2022
@jaimergp
Copy link
Contributor Author

Blocked by #505

@jaimergp jaimergp closed this Mar 14, 2022
@jaimergp jaimergp reopened this Mar 14, 2022
@jaimergp
Copy link
Contributor Author

jaimergp commented Mar 15, 2022

Local tests:

  • PKGs and SHs work on macOS
  • SHs work on Linux
  • EXEs work on Windows

@jaimergp
Copy link
Contributor Author

Windows needs fixing. Looks like the y-axis accumulator is not working as expected.

image

@jaimergp
Copy link
Contributor Author

Fixed locally! Screenshots of the different configs for register_python and initialize_conda, respectively:

With True / True (default):

image

With False / True:

image

With True / False:

image

With False / False:

image

@jaimergp jaimergp marked this pull request as ready for review March 23, 2022 14:21
@jaimergp jaimergp changed the title add initialize_conda and register_python keys to control whether to show these options (WIP) add initialize_conda and register_python keys to control whether to show these options Mar 25, 2022
larsoner
larsoner previously approved these changes Jul 6, 2022
@jaimergp jaimergp requested a review from a team as a code owner August 12, 2022 09:05
@jaimergp jaimergp merged commit b3ba67f into conda:main Aug 30, 2022
@hoechenberger
Copy link
Contributor

Amazing, thank you!!!

@github-actions github-actions bot added the locked [bot] locked due to inactivity label Aug 31, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 31, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla-signed [bot] added once the contributor has signed the CLA locked [bot] locked due to inactivity
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

5 participants