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

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

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
Copy link

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

  • The page pages.id/common/clang++.md has an outdated more info link.
  • The page pages.id/common/clang++.md has an outdated page title.
  • 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.

@@ -6,24 +6,28 @@

- Compile a source code file into an executable binary:
Copy link
Member

Choose a reason for hiding this comment

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

This line should be changed as well, since we are now compiling multiple source code files

@sebastiaanspeck
Copy link
Member

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

  • The page pages.id/common/clang++.md has an outdated more info link.
  • The page pages.id/common/clang++.md has an outdated page title.
  • 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.

@reinhart1010 can you fix this?

pages.id/common/clang++.md Outdated Show resolved Hide resolved
pages.id/common/clang++.md Outdated Show resolved Hide resolved
Co-authored-by: Sebastiaan Speck <12570668+sebastiaanspeck@users.noreply.github.com>
@tldr-bot
Copy link

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

  • The page pages.id/common/clang++.md has an outdated more info link.
  • 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.

@sebastiaanspeck
Copy link
Member

Will try to fix the more-info issue later

@tldr-bot
Copy link

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.

@sebastiaanspeck
Copy link
Member

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

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.

3 participants