Skip to content

Block running CI on style passing. #512

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
Draft
wants to merge 20 commits into
base: main
Choose a base branch
from
Draft

Block running CI on style passing. #512

wants to merge 20 commits into from
+158 −30

Conversation

@hughcars
Copy link
Collaborator

@hughcars hughcars commented Oct 1, 2025

No description provided.

hughcars added 20 commits October 1, 2025 13:46
@hughcars
Add path filter to only run CI as necessary
8d08f26
@hughcars
WIP
197b800
@hughcars
Update linux matrix
b4b296f
@hughcars
Fix matrix.config for other jobs, add macos
11127ee
@hughcars
Change to smaller runners for skip runs
7b5abdb
@hughcars
Fix indentation
df99fc7
@hughcars
Add | to if statements
9321680
@hughcars
Fix spack and singularity job names
ea68a76
@hughcars
Change filtering to be positive, test if a valid change occurs, and a…
16cd377 
…lways gen docs
@hughcars
Ensure action triggers if the action file is changed
9df3a13
@hughcars
Add dummy check-run so that a non-dependent run is present
762690a
@hughcars
Debugging
4fee990
@hughcars
WIP
ae167c2
@hughcars
Modify linux job to check the path-filter
ef32906
@hughcars
Try get linux to trigger again
4ac3a0f
@hughcars
Update mac
812453e
@hughcars
Add filtering to singularity and spack builds
5b7e55f
@hughcars
Use large palace runner in doc generation, given spack usage
b355b27
@hughcars
Delete path-filter.yml, moved filters into specific jobs
7c230ef
@hughcars
Only trigger if Style passes
f7a5690
@hughcars hughcars force-pushed the hughcars/restrict-github-ci-style-barrier branch from 161850d to f7a5690 Compare October 1, 2025 21:49
@hughcars hughcars force-pushed the hughcars/restrict-github-ci branch 2 times, most recently from d28905a to 81620fb Compare October 2, 2025 16:23
Base automatically changed from hughcars/restrict-github-ci to main October 2, 2025 17:35
@Sbozzolo
Copy link
Member

Sbozzolo commented Oct 3, 2025

I think this can be closed?

@hughcars
Copy link
Collaborator Author

hughcars commented Oct 3, 2025

I think this can be closed?

This was actually a slightly related issue, it makes a DAG with Style first. The issue is that you can't actually test this without pushing to main , so I hadn't checked it properly yet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@hughcars @Sbozzolo