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.

Sign up for GitHub

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

Jump to bottom

kill: update link #12054

Merged
merged 1 commit into from
Jan 10, 2024
Merged

kill: update link #12054

merged 1 commit into from
Jan 10, 2024

Conversation

sebastiaanspeck
Copy link
Member

  • 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.

Ref #12044

@github-actions github-actions bot added the translation Translate pages from one language to another. label Jan 10, 2024
@tldr-bot
Copy link

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

  • The page pages.es/common/kill.md is outdated, based on number of commands.
  • The page pages.fr/common/kill.md is outdated, based on number of commands.
  • The page pages.ko/common/kill.md is outdated, based on number of commands.
  • The page pages.pt_BR/common/kill.md is outdated, based on number of commands.

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

@kbdharun kbdharun merged commit 41b7426 into tldr-pages:main Jan 10, 2024
3 checks passed
tricantivu
tricantivu approved these changes Jan 10, 2024
View reviewed changes
Copy link
Member

@tricantivu tricantivu left a comment •
edited
Loading

Choose a reason for hiding this comment

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

The Spanish translation looks good to me. Although I would remove the prefix SIG from the signal SIGUSR1 because I think POSIX requires them to be like that.

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

@gutjuri gutjuri gutjuri approved these changes

@kbdharun kbdharun kbdharun approved these changes

@tricantivu tricantivu tricantivu approved these changes

@isaacvicente isaacvicente Awaiting requested review from isaacvicente isaacvicente is a code owner

@vitorhcl vitorhcl Awaiting requested review from vitorhcl vitorhcl is a code owner

@IMHOJEONG IMHOJEONG Awaiting requested review from IMHOJEONG IMHOJEONG is a code owner

@Nico385412 Nico385412 Awaiting requested review from Nico385412 Nico385412 is a code owner

@nicokosi nicokosi Awaiting requested review from nicokosi nicokosi is a code owner

@navarroaxel navarroaxel Awaiting requested review from navarroaxel navarroaxel is a code owner

@kant kant Awaiting requested review from kant kant is a code owner

Assignees
No one assigned
Labels
translation Translate pages from one language to another.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@sebastiaanspeck @tldr-bot @gutjuri @kbdharun @tricantivu