Skip to content

Conversation

jjhelmus
Copy link
Contributor

@jjhelmus jjhelmus commented Jun 5, 2025

Description

Do not activate an environment when initializing the shell integration in the dev script. This activation will use the default_activation_env environment name which may not exist in the bootstrap install causing the script to fail. The development base environment is explicitly activated in the next block.

Example behavior without this script with default_activation_env: work configured:

❯ . dev/start
Deactivating 1 environment(s)...
Update shell scripts...
Initializing shell integration...

EnvironmentNameNotFound: Could not find conda environment: work
You can list all discoverable environments with `conda info --envs`.


Error: failed to initialize shell integration

With this change:

❯ . dev/start
Deactivating 1 environment(s)...
Update shell scripts...
Initializing shell integration...
Activating devenv-3.10-c...

Checklist - did you ...

  • Add a file to the news directory (using the template) for the next release's release notes?
  • Add / update necessary tests?
  • Add / update outdated documentation?

Do not activate an environment when initializing the shell integration
in the dev script. This activation will use the `default_activation_env`
environment name which may not exist in the bootstrap install causing
the script to fail. The development base environment is explicitly
activated in the next block.
@jjhelmus jjhelmus marked this pull request as ready for review June 5, 2025 16:54
@jjhelmus jjhelmus requested a review from a team as a code owner June 5, 2025 16:54
@github-project-automation github-project-automation bot moved this to 🆕 New in 🔎 Review Jun 5, 2025
@conda-bot conda-bot added the cla-signed [bot] added once the contributor has signed the CLA label Jun 5, 2025
Copy link
Contributor

@kenodegard kenodegard left a comment

Choose a reason for hiding this comment

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

need similar change in dev/start.bat

@github-project-automation github-project-automation bot moved this from 🆕 New to 🏗️ In Progress in 🔎 Review Jun 5, 2025
@jjhelmus
Copy link
Contributor Author

jjhelmus commented Jun 5, 2025

need similar change in dev/start.bat

Done, although I have not tested the change on a live machine.

@github-project-automation github-project-automation bot moved this from 🏗️ In Progress to ✅ Approved in 🔎 Review Jun 5, 2025
@jjhelmus jjhelmus merged commit 8d6d044 into conda:main Jun 5, 2025
15 checks passed
@github-project-automation github-project-automation bot moved this from ✅ Approved to 🏁 Done in 🔎 Review Jun 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed [bot] added once the contributor has signed the CLA
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants