Skip to content

Conversation

soapy1
Copy link
Contributor

@soapy1 soapy1 commented Jul 2, 2025

Description

Follow up to #14937.

The new conda.model.environment.Environment class has a very similar name to the conda.env.env.Environment class. These classes often intermingle, and their similar names makes working with them confusing.

conda.model.environment.Environment represents a conda environment.
conda.env.env.Environment represents an environment.yaml file.

This PR proposes renaming conda.env.env.Environment -> conda.env.env.EnvironmentYaml, by deprecating conda.env.env.Environment and introducing a new class conda.env.env.EnvironmentYaml.

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?

@github-project-automation github-project-automation bot moved this to 🆕 New in 🔎 Review Jul 2, 2025
@conda-bot conda-bot added the cla-signed [bot] added once the contributor has signed the CLA label Jul 2, 2025
conda/env/env.py Outdated
with open(self.filename, "wb") as fp:
self.to_yaml(stream=fp)


@deprecated("26.3", "26.9")
Copy link
Member

Choose a reason for hiding this comment

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

Let's add an addendum here, explaining the change

Copy link

codspeed-hq bot commented Jul 2, 2025

CodSpeed Instrumentation Performance Report

Merging #14981 will not alter performance

Comparing soapy1:rename-environment-yaml (48b9b45) with main (9c502c7)

Summary

✅ 21 untouched benchmarks

@soapy1 soapy1 force-pushed the rename-environment-yaml branch 3 times, most recently from 7c83dfe to d940080 Compare July 2, 2025 17:12
@soapy1 soapy1 force-pushed the rename-environment-yaml branch from d940080 to 9e810e1 Compare July 2, 2025 21:10
@soapy1
Copy link
Contributor Author

soapy1 commented Jul 2, 2025

pre-commit.ci autofix

@soapy1 soapy1 marked this pull request as ready for review July 2, 2025 23:35
@soapy1 soapy1 requested a review from a team as a code owner July 2, 2025 23:36
@soapy1 soapy1 requested review from kenodegard and jezdez July 2, 2025 23:36
@github-project-automation github-project-automation bot moved this from 🆕 New to ✅ Approved in 🔎 Review Jul 3, 2025
@jezdez jezdez merged commit 7dbf0c9 into conda:main Jul 3, 2025
75 checks passed
@github-project-automation github-project-automation bot moved this from ✅ Approved to 🏁 Done in 🔎 Review Jul 3, 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.

4 participants