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.

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

make pkg/grammar free from any mission-control dependencies #1311

Merged
merged 2 commits into from
Feb 3, 2025

Conversation

adityathebe
Copy link
Member

resourceSelector.Matches() needs to use grammar.Parse but that creates an import cycle between types and grammar packages.

@adityathebe adityathebe requested a review from moshloop January 31, 2025 16:35
@adityathebe adityathebe force-pushed the chore/refactor-grammar-types branch from eb38138 to 03af041 Compare January 31, 2025 16:49
@moshloop moshloop enabled auto-merge (rebase) February 3, 2025 07:01
* feat: support grammar in resource selector .Matches
* feat: implement grammar in resource selector .Matches()
* properties lookup support for other operators
* chore: use generic fields matcher for all models
* fix: handle query field OR operator
@adityathebe adityathebe force-pushed the chore/refactor-grammar-types branch from 1b7eea9 to 5e33a6e Compare February 3, 2025 07:37
@moshloop moshloop merged commit 4ae473f into main Feb 3, 2025
6 checks passed
@moshloop moshloop deleted the chore/refactor-grammar-types branch February 3, 2025 07:41
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