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

[pt] Improved/rewrote rule ID:BÁSICO_ELEMENTAR #9984

Merged
merged 1 commit into from
Dec 27, 2023

Conversation

marcoagpinto
Copy link
Member

Heya, @susanaboatto and @p-goulart

I have rewritten the rule.

Now, instead of two subrules, all is done in one rule.

The words it accepts are there, making it easier to add or remove words.

This means, that if one of you detects false positives, it is just a matter of deleting the specific word from the rule.

Copy link
Collaborator

@p-goulart p-goulart left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A more cautious approach, but I think this is wiser in this case, esp. for academic-tagged rules.

@marcoagpinto marcoagpinto merged commit d1d8630 into master Dec 27, 2023
2 checks passed
@marcoagpinto marcoagpinto deleted the lt_marcoagpinto_20231223_1634 branch December 27, 2023 10:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants