Skip to content

Conversation

@stheoulle
Copy link
Contributor

@stheoulle stheoulle commented Dec 9, 2025

Checklist

  • The page(s) are in the correct platform directories: common, linux, osx, windows, sunos, android, etc.
  • The page description(s) have links to documentation or a homepage.
  • The page(s) follow the content guidelines.
  • The page(s) follow the style guide.
  • The PR contains at most 5 new pages.
  • The PR is authored by me, or has been human-reviewed if it was created with AI or machine translation software.
  • The PR title conforms to the recommended templates.
  • Version of the command being documented (if known):
  • Reference issue: #

@github-actions github-actions bot added the new translation PRs translating a page(s) from English to other languages. label Dec 9, 2025
@stheoulle stheoulle marked this pull request as draft December 9, 2025 13:17
@Managor Managor changed the title shell commons : add french translation shell syntax: add French translation Dec 9, 2025
@stheoulle
Copy link
Contributor Author

The more informations links are the same as the one used in the English translation.
The pages.fr/common/..md has no more information link, but the bot says otherwise

@stheoulle stheoulle marked this pull request as ready for review December 9, 2025 13:20
@Managor Managor changed the title shell syntax: add French translation !, $,%, ,, .: add French translation Dec 9, 2025
Copy link
Collaborator

@ivanbaluta ivanbaluta left a comment

Choose a reason for hiding this comment

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

Please note the French-Specific Rules regarding typography

Copy link
Collaborator

@ivanbaluta ivanbaluta left a comment

Choose a reason for hiding this comment

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

sebastiaanspeck and others added 4 commits December 10, 2025 05:45
Co-authored-by: Ivan Baluta <[email protected]>
Co-authored-by: Ivan Baluta <[email protected]>
Co-authored-by: Ivan Baluta <[email protected]>
Co-authored-by: Ivan Baluta <[email protected]>
@tldr-bot

This comment has been minimized.

Copy link
Member

@sebastiaanspeck sebastiaanspeck left a comment

Choose a reason for hiding this comment

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

Please read the French specific part in the Style Guide.

@ivanbaluta
Copy link
Collaborator

@stheoulle You applied the suggestion for the first file, but the same fix is needed for the other files as well.
Could you please update the remaining files?

@stheoulle
Copy link
Contributor Author

The other files should be good now

Copy link
Collaborator

@noraj noraj left a comment

Choose a reason for hiding this comment

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

LGTM excepts that placeholder should be translated too when possible.

@stheoulle stheoulle requested a review from noraj December 12, 2025 07:09
@Managor
Copy link
Member

Managor commented Dec 12, 2025

The page was recently updated. Please update the translation accordingly https://github.com/tldr-pages/tldr/blame/main/pages/common/!.md

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

new translation PRs translating a page(s) from English to other languages.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants