-
Notifications
You must be signed in to change notification settings - Fork 55
ci: add nox utilities for conditional session execution instead of using github actions #1982
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
Merged
Merged
Changes from 61 commits
Commits
Show all changes
74 commits
Select commit
Hold shift + click to select a range
6dfd3a1
Add nox session and utilities for conditionally running tests based o…
egparedes 5fb4a81
Add name matching mechanism for selecting which env variable are pass…
egparedes b032f35
Merge branch 'main' into new-conditionals
egparedes e24eaeb
Refactor to get conditional activation flag from the environment
egparedes bd66f86
WIP: fixes
egparedes eeb2709
More fixes
egparedes b945f52
Large refactoring moving all utilities to a different file
egparedes a74bdb6
Cleanups and fixes
egparedes c483e64
Test calling new nox way for eve from Github actions instead of fallback
egparedes f0d5c82
Cleanups and fixes
egparedes 3fd76e8
Fix
egparedes 4ece1f1
Try git diff changes
egparedes 3456cb6
Print to stderr in verbose mode
egparedes fcc242c
Put back "origin/"
egparedes 0a30b09
Add docstrigngs and fix verbose mode bug
egparedes 03275fb
Use lightweight launch venv with only nox
egparedes b3655ca
Improve docstrings
egparedes b7aa661
Merge branch 'main' into new-conditionals
egparedes a6eed30
Fix bug
egparedes e8f0497
Read python versions from file
egparedes 23980d7
Update noxfile_utils.py
egparedes a4be642
Update noxfile.py
egparedes 019cf6c
Use uvx for nox
egparedes 08ea3c5
Fixes
egparedes 3ea0c5e
Cleanups and fixes
egparedes 763f903
Adding unit tests
egparedes 0001431
Tests cleanups
egparedes d476732
Finish unit tests
egparedes 052bbaf
Cleanups and docs enhancements
egparedes bda11db
Update GitHub actions
egparedes ede2ad3
Missing edit from previous commit
egparedes 48c0dd7
Fix env var names
egparedes 7ca8037
Fix env vars prefix
egparedes dc06e36
Add utility to create env dicts and other cleanups
egparedes 61643d4
Merge branch 'main' into new-conditionals
egparedes 9c688f0
Add run_session util to create the env automatically
egparedes cd77635
Copilot review suggestions
egparedes 520acf8
Merge branch 'main' into new-conditionals
egparedes e9fcdfd
fix
egparedes 60ccc5b
Change development docs
egparedes 4c4937d
Fix style
egparedes afe48dd
Fix noxfile calls to run_session and other cleanups
egparedes 8d7ac17
Update nox version and remove unneeded env filtering feature from nox…
egparedes 3b8f38b
Fix unnecessary deep clones in actions and a bug in dev-tasks.py
egparedes 9477463
Merge branch 'main' into new-conditionals
egparedes 641d521
Add missing changes from previous commit
egparedes 7f7917a
Change import mechanism for noxfile_utils
egparedes 8802cc7
More cleanups, add is_affected session and update docs.
egparedes e604f44
Replace `uvx nox` by `uv run noxfile.py` in github actions, as suppor…
egparedes 3b6b25d
Doctrsing fixes
egparedes 983e73d
Move noxfile_utils tests from nox to pre-commit
egparedes 2312e0f
Add assert for safety
egparedes 19105af
Fix noxfile tests in pre-commit
egparedes 3d895d3
Update docs/development/tools/ci-infrastructure.md
egparedes ca0140f
Update docs/development/tools/ci-infrastructure.md
egparedes 27a4946
Final cleanups
egparedes 1911ab4
Get rid of custom NUM_PROCESSES variable and use standard PYTEST_XDIS…
egparedes c43de08
Remove unused git diff cache from noxfile_utils
egparedes 3cf2b4a
Fix typos
egparedes 72a1815
Merge branch 'main' into new-conditionals
egparedes c6429f4
More cleanups
egparedes cddca1a
Clean up pre-commit tests for noxfile_utils
egparedes 3f8e49d
Increase slurm time limit for cscs-ci
egparedes 8ed755a
Test CSCS-cI changes
egparedes 53e7bb4
Try again
egparedes 0124e4e
Another try following Andreas Fink tip
egparedes 2c252f3
Fix typos
egparedes 2033c6c
Add missing `jq` tools to base dockerfile
egparedes 05b8220
Merge branch 'main' into new-conditionals
egparedes 22bc05c
Add github remote in actual docker image
egparedes 0dbd2ff
More fixes
egparedes 21deb87
Add noxfile_utils.py to session path filters
egparedes c3453d5
Rename commit_spec variables
egparedes 67ac9e0
Remove unneeded list
egparedes File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.