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

Add Hebrew translations #488

Merged
merged 7 commits into from
Jun 1, 2024
Merged

Conversation

ronbentata
Copy link
Contributor

Summary

Added an Hebrew translation + ReadMe

Translation checklist

  • I have ran the application to make sure my code runs
  • I have used the correct ISO 639-1 code for my translations (e.g. en)
  • I have made sure to leave a , at the end of each translation added
  • I have added the language to supportedLanguages in alphabetical order
  • I have added the language to the table in README.md in alphabetical order
  • (If you translated the README) I have translated the README and linked it in the main README.md

README-Hebrew.md Outdated Show resolved Hide resolved
auto_py_to_exe/web/js/i18n.js Outdated Show resolved Hide resolved
auto_py_to_exe/web/js/i18n.js Outdated Show resolved Hide resolved
auto_py_to_exe/web/js/i18n.js Show resolved Hide resolved
auto_py_to_exe/web/js/i18n.js Outdated Show resolved Hide resolved
- adjust format.
- fix translationMap key.
@ronbentata
Copy link
Contributor Author

please update me regarding issue 1 and 4 (you mentioned and I responded).
if it's ok with you, I'll push the latest updates

- add Hebrew ref to Trans table in main md.
- format i18n.
- rearrange He before Hi translations.
@ronbentata
Copy link
Contributor Author

pushed new commit to resolve latest mentioned issues

Copy link
Owner

@brentvollebregt brentvollebregt left a comment

Choose a reason for hiding this comment

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

The formatting check failed again, here is the output from the action run:

Checking formatting...
[warn] auto_py_to_exe/web/js/i18n.js
[warn] README-Hebrew.md
[warn] README.md
[warn] Code style issues found in 3 files. Forgot to run Prettier?
Problem running prettier with --check .

auto_py_to_exe/web/js/i18n.js Outdated Show resolved Hide resolved
@brentvollebregt
Copy link
Owner

Hey @ronbentata, not sure if you saw my couple of comments about formatting - this still needs to be sorted before merge.

@ronbentata
Copy link
Contributor Author

run prettier again to fix format on specific files

Copy link
Owner

@brentvollebregt brentvollebregt left a comment

Choose a reason for hiding this comment

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

Cheers, one more thing I spotted when I ran the application myself. After this, it will be good to merge 👍

auto_py_to_exe/web/js/i18n.js Outdated Show resolved Hide resolved
@brentvollebregt brentvollebregt changed the title add Hebrew translation Add Hebrew translations May 31, 2024
@brentvollebregt brentvollebregt merged commit 4f2fb2d into brentvollebregt:master Jun 1, 2024
1 check passed
@brentvollebregt
Copy link
Owner

Thanks for the contribution! This has been released in v2.44.0 🚀

@ronbentata
Copy link
Contributor Author

ronbentata commented Jun 5, 2024 via email

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