Skip to content

Update rsactftool.py.md #16176

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Update rsactftool.py.md #16176

wants to merge 1 commit into from

Conversation

kant
Copy link
Member

@kant kant commented Apr 15, 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 page edit Changes to an existing page(s). review needed Prioritized PRs marked for reviews from maintainers. labels Apr 15, 2025
Copy link
Member

@MachiavelliII MachiavelliII left a comment

Choose a reason for hiding this comment

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

I reviewed the main page, and I didn't include the .py extension in the English page:
#16078

@kant
Copy link
Member Author

kant commented Apr 15, 2025

I follow the warning from here
Inconsistency found in file: pages.es/common/rsactftool.py.md: rsactftool should be rsactftool.py

@gutjuri
Copy link
Member

gutjuri commented Apr 15, 2025

Not sure if people looking for the page would intuitively search for rsactftool instead of rsactftool.py. So it might be worth it to add an alias page

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

Successfully merging this pull request may close these issues.

3 participants