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] Changed name and improved rule ID:ARREAR_ARRIAR #9934

Merged
merged 1 commit into from
Dec 18, 2023

Conversation

marcoagpinto
Copy link
Member

Heya,

I have made the rule more useful.

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.

Rule logic is okay, but I personally feel like for such terms it might be worth it to have the '(Possível) confusão de termos' message as the short message and a longer message that actually makes reference to the meaning of the verbs?

@marcoagpinto
Copy link
Member Author

Rule logic is okay, but I personally feel like for such terms it might be worth it to have the '(Possível) confusão de termos' message as the short message and a longer message that actually makes reference to the meaning of the verbs?

I am brain toasted 😋 I can't think of good messages.

@marcoagpinto marcoagpinto merged commit 7e13aaa into master Dec 18, 2023
2 checks passed
@marcoagpinto marcoagpinto deleted the lt_marcoagpinto_20231216_2342 branch December 18, 2023 10:04
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