-
Notifications
You must be signed in to change notification settings - Fork 190
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
ci: adds fixed env ci #1183
base: main
Are you sure you want to change the base?
ci: adds fixed env ci #1183
Conversation
Tests will fail since the env files need to be created initially. But it looks like they are not running through anyways? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Many thanks and this seems a great contribution!
This simplifies a lot the maintenance of the model indeed
I'd also recommend adding here a release note also to summarize the great three contributions you proposed :) you can find it in doc/release_notes.rst.
Many thanks :D
Thank you and no worries! I added release notes for all three PRs. The Tests CI fails right now, so this needs another look once this one is merged |
Adds pinned env ci and runs test on pinned env instead, as implemented in pypsa-eur.
Removes
envs/environment.fixed.yaml
and addsenvs/<os>-pinned.yaml
instead which are automatically updated.This is useful for the devcontainer implementation by @drifter089 and @GbotemiB
See PyPSA/pypsa-eur#1379