Skip to content

Conversation

@marcos-aparicio
Copy link
Contributor

  • 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 page(s) follow the style guide.
  • The PR title conforms to the recommended templates.
  • Version of the command being documented (if known):

@github-actions github-actions bot added the new translation PRs translating a page(s) from English to other languages. label Mar 28, 2025
@tldr-bot
Copy link

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

  • The page pages.es/common/hledger.md is outdated, based on the command contents itself, compared to the English page.

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

@marcos-aparicio
Copy link
Contributor Author

For any reviewers seeing this, the tldr-bot error is indeed intended, I've checked and it happens because the command string itself is different in the Spanish version in comparison to the English version. That is because the Spanish version includes some translations within the command itself as those sections do not necessarily need to be in English despite not having the {{}} brackets. The following image might be more descriptive.

image

@sebastiaanspeck
Copy link
Member

For any reviewers seeing this, the tldr-bot error is indeed intended, I've checked and it happens because the command string itself is different in the Spanish version in comparison to the English version. That is because the Spanish version includes some translations within the command itself as those sections do not necessarily need to be in English despite not having the {{}} brackets. The following image might be more descriptive.

image

Please revert the non-placeholder translations back to English.

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.

See my comment above

@Managor Managor merged commit acda299 into tldr-pages:main Mar 29, 2025
3 checks passed
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.

5 participants