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

Fix Validation error for rule_modifier in grammar.y #2019

Merged
merged 2 commits into from
Dec 7, 2023

Conversation

tibibyte
Copy link
Contributor

@tibibyte tibibyte commented Dec 1, 2023

Prevent user from using the rule modifiers 'global' and 'private' more than once while also allowing to use them together in any order.

rule_modifiers 'private' and 'global' can no longer be used more than once in a rule definition. They can be used exclusively or together in any order.
Copy link

google-cla bot commented Dec 1, 2023

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@plusvic plusvic merged commit 2d103d3 into VirusTotal:master Dec 7, 2023
8 of 10 checks passed
plusvic added a commit that referenced this pull request Dec 10, 2023
This reverts commit 2d103d3. The previous fix introduces shit-reduce and reduce-reduce conflicts in the grammar. The grammar.y file was not correctly processed by `bison`.
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