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

Cleanup of conda-incubator & replace deprecated pkg_resources #1045

Merged
merged 2 commits into from
Mar 23, 2023

Conversation

nikfilippas
Copy link
Contributor

@nikfilippas nikfilippas commented Mar 22, 2023

This PR removes the python-version argument from the incubator. The Python version is specified in environment.yml. There is no need to pass it to the incubator, and in fact this is now trying to access a variable that doesn't exist.

It also replaces the deprecated module pkg_resources with the built-in importlib to assign pyccl.__version__ in __init__.py. The deprecated package was throwing warnings in testing.

@coveralls
Copy link

coveralls commented Mar 22, 2023

Pull Request Test Coverage Report for Build 4494008683

  • 3 of 4 (75.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 97.857%

Changes Missing Coverage Covered Lines Changed/Added Lines %
pyccl/init.py 3 4 75.0%
Totals Coverage Status
Change from base Build 4477013263: 0.0%
Covered Lines: 5480
Relevant Lines: 5600

💛 - Coveralls

@nikfilippas nikfilippas changed the title GHA: Remove leftover Python version in conda-incubator Cleanup of conda-incubator & replace deprecated pkg_resources Mar 22, 2023
@damonge damonge merged commit d514c13 into master Mar 23, 2023
@damonge damonge deleted the gha_miniconda_matrixpy branch March 23, 2023 14:00
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