Skip to content

Fix CI path excludes#4095

Merged
Earlopain merged 2 commits intoruby:mainfrom
Earlopain:fix-ci-paths
Apr 29, 2026
Merged

Fix CI path excludes#4095
Earlopain merged 2 commits intoruby:mainfrom
Earlopain:fix-ci-paths

Conversation

@Earlopain
Copy link
Copy Markdown
Collaborator

@Earlopain Earlopain commented Apr 29, 2026

They need wildcards, otherwise they don't match.
Previously it always ran on PRs but in https://github.com/ruby/ruby-prism/commit/c4d202dc4866d6f03cc6cbda083a8950c6e6e42e I also applied it to PRs.
That means that previously it didn't run on main only.

This fixes this.

They need wildcards, otherwise they don't match.
Previously it always ran on PRs but in github.com/ruby/ruby-prism/commit/c4d202dc4866d6f03cc6cbda083a8950c6e6e42e I also applied
it it PRs.
That means that previously it didn't run on main.

This fixes this
@Earlopain Earlopain marked this pull request as ready for review April 29, 2026 09:56
@Earlopain
Copy link
Copy Markdown
Collaborator Author

Earlopain commented Apr 29, 2026

#4093 bumped JRuby but it fails CI (didn't actually run) so I reverted it (cc @andreaTP
https://github.com/ruby/prism/actions/runs/25101220259/job/73550648784)

@Earlopain Earlopain merged commit c9c1bbb into ruby:main Apr 29, 2026
69 checks passed
@andreaTP
Copy link
Copy Markdown
Contributor

Thanks for the heads up, cc. @enebo @headius I think it might be on their plate

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants