这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@jaimergp
Copy link
Contributor

@jaimergp jaimergp commented Mar 12, 2025

Description

Instead of relying on ${CONDA_DEFAULT_ENV} directly, use it via the context system.

Make sure this is used in conda activate too, instead of the hardcoded base instance there.

This would allow installers to ship a non-base default environment that can be auto activated on shell initialization, preventing users from unintentionally installing things in base.

Comes from #11342 (comment).

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?

@codspeed-hq
Copy link

codspeed-hq bot commented Mar 12, 2025

CodSpeed Performance Report

Merging #14666 will not alter performance

Comparing jaimergp:default-env (85566e0) with main (bc5fe62)

Summary

✅ 21 untouched benchmarks

@jaimergp jaimergp marked this pull request as ready for review March 17, 2025 10:25
@jaimergp jaimergp requested a review from a team as a code owner March 17, 2025 10:25
@jaimergp jaimergp added the build::review trigger a build for this PR label Mar 17, 2025
@jaimergp jaimergp changed the title Define context.default_env Define context.default_env (fork) Mar 17, 2025
@jaimergp
Copy link
Contributor Author

Reopening from branch in #14678

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build::review trigger a build for this PR 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.

2 participants