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 Tamil translation #65

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Add Tamil translation #65

wants to merge 2 commits into from

Conversation

Aeden-B
Copy link

@Aeden-B Aeden-B commented Jan 5, 2025

A few comments on the changes made in addition to adding the Tamil language:

  • I switched source.fixAll.eslint to string "always", VS code expects to have ‘always’, ‘explicit’ or ‘never’ as value

  • I've corrected the readme that was out of date since make was removed

  • On Windows, I haven't managed to run the tests other than with the command I modified.

  • lastly, there are currently 3 KO tests for languages other than Tamil

    1. i18n for iso 639-1
      completeness
      ar completeness
      complete (to less):
      AssertionError [ERR_ASSERTION]: missing entry for bh
  1. i18n for iso 639-1
    completeness
    he completeness
    complete (to less):
    AssertionError [ERR_ASSERTION]: missing entry for aa

  2. i18n for iso 639-1
    completeness
    ko completeness
    complete (to less):
    AssertionError [ERR_ASSERTION]: missing entry for bh

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.

1 participant