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

[internal] Ignore empty rule sets. #17580

Merged
merged 1 commit into from
Nov 18, 2022
Merged

[internal] Ignore empty rule sets. #17580

merged 1 commit into from
Nov 18, 2022

Conversation

kaos
Copy link
Member

@kaos kaos commented Nov 18, 2022

Motivation being to ease incremental adoption, where rules may be filtered through some applicability function.

@kaos kaos added the category:internal CI, fixes for not-yet-released features, etc. label Nov 18, 2022
@kaos kaos requested a review from stuhood November 18, 2022 16:16
@kaos kaos merged commit 7583d1e into pantsbuild:main Nov 18, 2022
@kaos kaos deleted the dep-rules-ignore-empty branch November 18, 2022 18:18
@stuhood stuhood mentioned this pull request Nov 19, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@stuhood stuhood stuhood approved these changes

Assignees
No one assigned
Labels
category:internal CI, fixes for not-yet-released features, etc.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@kaos @stuhood