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

cc, g++, gcc: add Tamil translation #8292

Merged
merged 7 commits into from
Aug 5, 2022
Merged

Conversation

kbdharun
Copy link
Member

@kbdharun kbdharun commented Aug 3, 2022

  • The page(s) are in the correct platform directories: common, linux.
  • The page(s) have at most 8 examples.
  • The page description(s) have links to documentation or a homepage.
  • The page(s) follow the content guidelines.
  • The PR title conforms to the recommended templates.

@github-actions github-actions bot added the translation Translate pages from one language to another. label Aug 3, 2022

- பல மூல கோப்புகளை இயங்கக்கூடியதாக தொகுக்கவும்:

`gcc {{பாதை /க்கு / மூல1.c பாதை /க்கு / மூல2.c ...}} --output {{பாதை/க்கு/வெளியீடு_இயங்கக்கூடியது}}`
Copy link
Collaborator

Choose a reason for hiding this comment

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

In https://github.com/tldr-pages/tldr/blob/main/contributing-guides/translation-templates/common-arguments.md we have the Tamil translation கோப்பு for path/to/file. Should we use this or your version?

Copy link
Member Author

Choose a reason for hiding this comment

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

கோப்பு/டு/மூல1.c would become file/to/source.c and but in this case, we are using path to source, because path means பாதை, and கோப்பு means file. As mentioned in Link , in this case I think path to source would better align with the main pages.

Source means மூல in Tamil so I had used பாதை /க்கு / மூல1.c , I will update this to பாதை /டு /மூல.c for better readability.

Copy link
Collaborator

Choose a reason for hiding this comment

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

So do you think we should update our common-arguments.md translation guide?

Copy link
Member Author

@kbdharun kbdharun Aug 4, 2022

Choose a reason for hiding this comment

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

So do you think we should update our common-arguments.md translation guide?

Yeah, I think it would be better to update it. @marchersimon

image
image

As அடைவு means directory only and not path\to\directory and கோப்பு means file only not path\to\file . So we might need to update these 2 commands and the existing referring pages. Also, In some pages like this one they use அடைவிற்குப்/பாதை which is the right word for path/to/directory.

Suggestions:

  1. Update path/to/directory from அடைவு toஅடைவிற்குப்/பாதை.
  2. Update path/to/file from கோப்பு to something like பாதை/டு/கோப்பு(path/to/file) or கோப்பின்/பாதை (file\path).
  3. We can apply the changes to 3 rd command/column to பாதை/டு/கோப்பு_அல்லது_அடைவு from கோப்போ/அடைவோ. Edit: I am not sure about this 3rd one, as it is mentioned path\to\file_or_directory so I need to figure out a way to merge the above 2 words. Figured it out 🙂

Update: Created pull request (#8301)

@navarroaxel navarroaxel changed the title cc,gcc,g++: add Tamil translation cc, g++, gcc: add Tamil translation Aug 4, 2022
Copy link
Collaborator

@marchersimon marchersimon left a comment

Choose a reason for hiding this comment

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

👍🏽

@navarroaxel navarroaxel merged commit 6459d45 into tldr-pages:main Aug 5, 2022
@kbdharun kbdharun deleted the translate2 branch August 5, 2022 11:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
translation Translate pages from one language to another.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants