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

clang, clang++, gcc, g++: standardize English page and Indonesian translation #13751

Merged
merged 10 commits into from
Sep 23, 2024

Conversation

reinhart1010
Copy link
Collaborator

  • The page(s) are in the correct platform directories: common, linux, osx, windows, sunos, android, etc.
  • 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.
  • Version of the command being documented (if known):

This commit adds the short and long-form parameters, enforce imperative mood, and standardizes the contents between clang and gcc as well as clang++ and g++.

… translation

This commit adds the short and long-form parameters, enforce imperative mood, and standardizes the contents between `clang` and `gcc` as well as `clang++` and `g++`.
@github-actions github-actions bot added page edit Changes to an existing page(s). translation Translate pages from one language to another. labels Sep 19, 2024
@tldr-bot

This comment was marked as resolved.

pages/common/clang++.md Outdated Show resolved Hide resolved
@sebastiaanspeck

This comment was marked as resolved.

pages.id/common/clang++.md Outdated Show resolved Hide resolved
pages.id/common/clang++.md Outdated Show resolved Hide resolved
@tldr-bot

This comment was marked as resolved.

@sebastiaanspeck
Copy link
Member

Will try to fix the more-info issue later

@tldr-bot

This comment was marked as resolved.

@sebastiaanspeck

This comment was marked as resolved.

@reinhart1010
Copy link
Collaborator Author

Hello! I've noticed something unusual when checking this PR:

  • The page pages.id/common/clang.md is outdated, based on the command contents itself.
  • The page pages.id/common/g++.md is outdated, based on number of commands.

Is this intended? If so, just ignore this comment. Otherwise, please double-check the commits.

If you fix this, the PR is good to go

Done

@tldr-bot

This comment was marked as resolved.

pages.id/common/gcc.md Outdated Show resolved Hide resolved
pages.id/common/gcc.md Outdated Show resolved Hide resolved
pages.id/common/clang++.md Outdated Show resolved Hide resolved
pages.id/common/clang++.md Outdated Show resolved Hide resolved
pages.id/common/clang.md Outdated Show resolved Hide resolved
pages.id/common/g++.md Outdated Show resolved Hide resolved
pages.id/common/g++.md Outdated Show resolved Hide resolved
Co-authored-by: Sebastiaan Speck <[email protected]>
@tldr-bot

This comment was marked as resolved.

@tldr-bot

This comment was marked as resolved.

@tldr-bot

This comment was marked as resolved.

Copy link
Member

@kbdharun kbdharun left a comment

Choose a reason for hiding this comment

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

LGTM, Thanks for your contribution.

pages/common/clang++.md Outdated Show resolved Hide resolved
pages/common/clang++.md Outdated Show resolved Hide resolved
pages/common/g++.md Outdated Show resolved Hide resolved
pages/common/g++.md Outdated Show resolved Hide resolved
pages.id/common/g++.md Outdated Show resolved Hide resolved
pages.id/common/g++.md Outdated Show resolved Hide resolved
pages.id/common/g++.md Outdated Show resolved Hide resolved
pages.id/common/g++.md Outdated Show resolved Hide resolved
pages.id/common/clang++.md Outdated Show resolved Hide resolved
@reinhart1010 reinhart1010 merged commit d29375d into tldr-pages:main Sep 23, 2024
4 checks passed
@1010bots 1010bots deleted the indonesian-11 branch September 24, 2024 05:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
page edit Changes to an existing page(s). translation Translate pages from one language to another.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants