-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
[be] Fix translations and mistakes. Add new rules #8409
Conversation
PlagaMedicum
commented
May 5, 2023
- Updated translations
- Added some rules in grammar
- Added ignore list
- Added Specific Case rule
- Added Simple Replace rule
- Updated translations - Added some rules in grammar - Added ignore list - Added Specific Case rule - Added Simple Replace rule
...-language-modules/be/src/main/java/org/languagetool/rules/be/BelarusianSpecificCaseRule.java
Outdated
Show resolved
Hide resolved
...-language-modules/be/src/main/java/org/languagetool/rules/be/BelarusianSpecificCaseRule.java
Outdated
Show resolved
Hide resolved
languagetool-language-modules/be/src/main/java/org/languagetool/rules/be/SimpleReplaceRule.java
Outdated
Show resolved
Hide resolved
...agetool-language-modules/be/src/main/resources/org/languagetool/MessagesBundle_be.properties
Outdated
Show resolved
Hide resolved
Fixed issues above. |
Gtihub says "This branch cannot be rebased due to conflicts", could you update the PR? |
Try it now |
Github still complains, see #8409 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It thinks that the file was modified, I guess
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I missed a new line character. Now everything should be fine, I hope it is fine now.
If it doesn't, how can I undo my commits on this file?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've used a merge commit this time, it worked with that.