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.

Sign up for GitHub

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

Jump to bottom

Look into only running CI tests associated with the code changes #90

Closed
MialLewis opened this issue Dec 20, 2023 · 0 comments · Fixed by #124
Closed

Look into only running CI tests associated with the code changes #90

MialLewis opened this issue Dec 20, 2023 · 0 comments · Fixed by #124
Assignees
@GuiMacielPereira
Labels
maintenance vesuvio-calibration vesuvio-package

Comments

@MialLewis
Copy link
Collaborator

Currently tests for both the analysis script and calibration script are run on every PR. Ideally, we should skip either the analysis or calibration script is no changes related to those scripts are made.
@GuiMacielPereira GuiMacielPereira self-assigned this Jun 28, 2024
@GuiMacielPereira GuiMacielPereira linked a pull request Jul 8, 2024 that will close this issue
Select analysis or calibration tests according to changes #120
Closed
@GuiMacielPereira GuiMacielPereira mentioned this issue Jul 8, 2024
Closed
This was linked to pull requests Jul 8, 2024
Add if statement to select tests #121
Closed
Add job for comparing changes #122
Closed
Add step for checking which files changed #123
Closed
@GuiMacielPereira GuiMacielPereira linked a pull request Jul 22, 2024 that will close this issue
Change github workflow to choose which tests to run #124
Merged
@GuiMacielPereira GuiMacielPereira mentioned this issue Jul 22, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@GuiMacielPereira GuiMacielPereira

Labels
maintenance vesuvio-calibration vesuvio-package
Projects
Release 6.11 Vesuvio
Status: Done
Milestone
No milestone
2 participants
@GuiMacielPereira @MialLewis