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

Remove --use-feature=2020-resolver command line option for obsolete pip 2020 solver #2493

Merged
merged 2 commits into from
Jan 24, 2022

Conversation

valeriupredoi
Copy link
Contributor

@valeriupredoi valeriupredoi commented Jan 20, 2022

Description

Pip introduced a more robust but different dependency resolver back when were were all working from home (sneaky! 😁 ), see communication release and that was breaking some stuffs back then, now it's the default resolver and the call to use the older resolver is deprecated and will introduce an error if used 👍 🍺

Just like in ESMValGroup/ESMValCore#1357

Tests all ok bar one related to pymon-journal that @zklaus has already flagged


Before you get started

Checklist

It is the responsibility of the author to make sure the pull request is ready to review. The icons indicate whether the item will be subject to the 🛠 Technical or 🧪 Scientific review.

@valeriupredoi valeriupredoi changed the title Remove --use-2020solver option for pip 2020 solver Remove --use-feature=2020-resolver command line option for obsolete pip 2020 solver Jan 20, 2022
Copy link

@zklaus zklaus left a comment

Choose a reason for hiding this comment

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

Curious that this appears only in one of the lines of run-tests-monitor script.
Could you do the same for Core?

@valeriupredoi
Copy link
Contributor Author

cheers @zklaus - I did that for Core already, we merged it last week. TBH no idea why it's only for one lonely test 😁

@valeriupredoi valeriupredoi merged commit 39368da into main Jan 24, 2022
@valeriupredoi valeriupredoi deleted the remove_2020solver branch January 24, 2022 14:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants