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

BUG: Resolves issue #364 #365

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

mforbes
Copy link

@mforbes mforbes commented Feb 6, 2022

No description provided.

@AlbertDeFusco
Copy link
Collaborator

Oh! I didn't know this variant exists. If you add it to the conda.recipe/meta.yaml and environment.yml files I wonder if the CI tests pass.

https://anaconda.org/main/ruamel.yaml

@mforbes
Copy link
Author

mforbes commented Feb 6, 2022

I added this. If it works, we might also be able to clean up the import cruft in yaml_file.py

@mforbes
Copy link
Author

mforbes commented Feb 7, 2022

Not sure why tests are failing. I.e. 3.8 on Mac OS X anaconda_project/internal/test/test_conda_api.py::test_pip_installed works for me. I think it might be related to provisioning - the CI is pulling in an old version of conda and pip for example.

@mbargull
Copy link

mbargull commented Nov 1, 2023

FYI, I've done the same in gh-399 while also removing the then not needed fallback to ruamel_yaml.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants