-
Notifications
You must be signed in to change notification settings - Fork 85
Add Romanian language support #848
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
base: master
Are you sure you want to change the base?
Conversation
|
Wow, that was unexpected ! Thanks a lot for the interest and effort, this is really appreciated. Since we are a small team developing and maintaining the software, I must add a few preliminary words of caution though:
With this out of the way, it seems you have been pretty thorough in your patch and we'll be happy to merge it -- one change I would request would be to remove the Romanian language tests from the main code base, since that's automatically included in our CI checks while we won't be able to maintain them. |
9a436e9 to
ff98ed9
Compare
|
The tests have been removed, didn't realize they are automatically included. Of course, I will make sure to maintain the Romanian implementation, breaking changes notwithstanding. |
890230c to
3c0168a
Compare
|
(don't worry about the CI failures, we're updating our docker images and runners, I'll merge as soon as it's restored) |
|
Sorry about the lack of follow-up ; we have been making some breaking changes in the lexer/parser (for the better, for example no patch should be needed in |
|
Sure, not a problem. Make all the breaking changes as needed. Sorry I cannot help you with more in depth PRs on the codebase itself, still much to learn. |
|
Hi! Sorry for the delay ; the syntax changes should be stabilised now -- the translated keywords should now only appear in two places:
|
44f2aa4 to
dd97f30
Compare
dd97f30 to
d596a1a
Compare
|
I have updated the branch. Running In
This sed command is supposed to extract command names from clerk's help output, but it's failing and returning nothing. When the extraction fails:
Old command (GNU-only):
New command (Portable):
|
|
Thanks a lot! The hand on making our build script portable is very much appreciated too. |
Description
This PR adds full Romanian language support to Catala. The implementation follows the existing pattern for multi-language support and includes all necessary components for a complete language integration.
--language=ro)compiler/desugared/name_resolution.mldoc/syntax/syntax_ro.catala_ro)syntax_highlighting/ro/)Checklist
If this PR adds a feature or has breaking changes
If this PR contains syntax changes
I confirm that have have checked and updated each of the following items if this PR impacts them:
doc/syntax/syntax_*.catala_*anddoc/syntax/catala_*.typ.syntax_highlighting/(all of them).tree-sittergrammar: