Skip to content

Conversation

Sarijen
Copy link
Collaborator

@Sarijen Sarijen commented Oct 6, 2025

  • 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 new translation PRs translating a page(s) from English to other languages. review needed Prioritized PRs marked for reviews from maintainers. labels Oct 6, 2025
Copy link
Member

@dmmqz dmmqz left a comment

Choose a reason for hiding this comment

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

Please update this for the English page as well.

@tldr-bot
Copy link

tldr-bot commented Oct 6, 2025

Hello! I've noticed something unusual when checking this PR at commit
8ef4016c2012048438dd8ddd780f6788d4294b73:

  • The page pages.cs/common/st-flash.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.

@Managor
Copy link
Member

Managor commented Oct 7, 2025

Fix bot issues.

@ali90h ali90h merged commit 6d1f27c into tldr-pages:main Oct 7, 2025
3 checks passed
@Managor
Copy link
Member

Managor commented Oct 7, 2025

@ali90h The bot issues hadn't been fixed

ali90h added a commit that referenced this pull request Oct 7, 2025
@ali90h
Copy link
Collaborator

ali90h commented Oct 7, 2025

oh, i'm sorry
Do I do a revert?

@Managor
Copy link
Member

Managor commented Oct 7, 2025

It'll have to stay. Someone will pick the mistake up or not.

@dmmqz
Copy link
Member

dmmqz commented Oct 7, 2025

I think this was fixed in #18521 @Managor

@Managor
Copy link
Member

Managor commented Oct 7, 2025

It wasn't. #18521 got merged before 8ef4016 I think

@Sarijen
Copy link
Collaborator Author

Sarijen commented Oct 7, 2025

I pushed the fix here before #18521 was merged if i recall correctly

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. review needed Prioritized PRs marked for reviews from maintainers.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants