Skip to content

Conversation

@blaginin
Copy link
Collaborator

Which issue does this PR close?

Related to #6880

Rationale for this change

When we were testing merge queues for this repo, I made all "rust test" steps mandatory. However, those "rust tests" weren't running for PRs that change documentation only - and as a result it was impossible to merge those PRs: #17183 (comment)

GitHub doesn't have a way to pair merge queues with paths-ignore: https://github.com/orgs/community/discussions/45899 - so I had to move those checks to a separate step (this is how this problem is solved by the community).

I'm using dorny/paths-filter as it has already been approved by ASF and it is desired for us to use it.

You'll notice predicate-quantifier is marked as an unsupported arg:

image

This arg is actually used, just not defined in the action definition. Once a release with dorny/paths-filter#279 is created, we'll switch to the new version and the warning will go away.

What changes are included in this PR?

A new action, rust-required-checks, that will become mandatory (requirement to enable merge queues):

  • When there are rust changes, it'll be "green" only when all the previous steps are green
  • When there are no rust changes, it gets green right away so we won't have to wait

Are these changes tested?

Yes:

Are there any user-facing changes?

No

@github-actions github-actions bot added the development-process Related to development process of DataFusion label Jan 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

development-process Related to development process of DataFusion

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant