Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

exo*: add pages #16405

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

Merged
merged 4 commits into from
May 28, 2025
Merged

exo*: add pages #16405

merged 4 commits into from
May 28, 2025

Conversation

davidlazch
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 mass changes Changes that affect multiple pages. new command Issues requesting creation of a new page or PRs adding a new page for a command. review needed Prioritized PRs marked for reviews from maintainers. labels May 7, 2025
@tldr-bot
Copy link

tldr-bot commented May 7, 2025

The build for this PR failed with the following error(s):

pages/common/exo-dbaas.md:
Error: Parse error on line 4:
...om/community/dbaas/>- List available D
-----------------------^
Expecting 'END_INFORMATION_LINK_URL', got 'NEWLINE'
pages/common/exo-dbaas.md:4: TLDR011 Page never contains more than a single empty line
pages/common/exo-iam.md:
Error: Parse error on line 4:
....com/community/iam/>- List all of the 
-----------------------^
Expecting 'END_INFORMATION_LINK_URL', got 'NEWLINE'
pages/common/exo-iam.md:4: TLDR011 Page never contains more than a single empty line
pages/common/exo-storage.md:
Error: Parse error on line 4:
.../community/storage/>- Create a new SOS
-----------------------^
Expecting 'END_INFORMATION_LINK_URL', got 'NEWLINE'
pages/common/exo-storage.md:4: TLDR011 Page never contains more than a single empty line

Please fix the error(s) and push again.

pages/common/exo-compute-block-storage.md Outdated Show resolved Hide resolved
pages/common/exo-compute-instance.md Outdated Show resolved Hide resolved
pages/common/exo-compute-sks.md Outdated Show resolved Hide resolved
pages/common/exo-compute.md Outdated Show resolved Hide resolved
pages/common/exo-dbaas.md Outdated Show resolved Hide resolved
pages/common/exo-iam.md Outdated Show resolved Hide resolved
pages/common/exo-storage.md Outdated Show resolved Hide resolved
pages/common/exo.md Outdated Show resolved Hide resolved
@Managor Managor changed the title exo*: Add page and alias pages exo*: add pages May 8, 2025
@Managor
Copy link
Collaborator

Managor commented May 8, 2025

Not sure what you meant by alias pages.

pages/common/exo-compute-instance.md Show resolved Hide resolved
Copy link
Member

@CleanMachine1 CleanMachine1 left a comment

Choose a reason for hiding this comment

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

Great PR!

pages/common/exo-compute-block-storage.md Outdated Show resolved Hide resolved
pages/common/exo-compute-block-storage.md Outdated Show resolved Hide resolved
pages/common/exo-compute-instance.md Outdated Show resolved Hide resolved
pages/common/exo-compute-sks.md Outdated Show resolved Hide resolved
pages/common/exo-dbaas.md Show resolved Hide resolved
pages/common/exo-dbaas.md Outdated Show resolved Hide resolved
pages/common/exo.md Outdated Show resolved Hide resolved
@davidlazch
Copy link
Contributor Author

Not sure what you meant by alias pages.

Found it a bit unclear for the naming convention when there are mass changes for a single command along with its sub commands.

In the templates, as I've understood, if its a sub command of a command its an alias page:

  • For a new page addition: ls: add page, docker-container-rm: add alias page

@sebastiaanspeck
Copy link
Member

sebastiaanspeck commented May 8, 2025

Not sure what you meant by alias pages.

Found it a bit unclear for the naming convention when there are mass changes for a single command along with its sub commands.

In the templates, as I've understood, if its a sub command of a command its an alias page:

  • For a new page addition: ls: add page, docker-container-rm: add alias page

An alias page is a page like this: https://github.com/tldr-pages/tldr/blob/main/pages/common/docker-container-rm.md?plain=1. It tells the user it should look to docker rm. For a subcommand (like docker rm) the title would be docker rm: add page. Hope this clarifies things

@Managor
Copy link
Collaborator

Managor commented May 8, 2025

Correction: docker-rm: add page The title reflects the filename.

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.

Please replace all {{<digits}} with <digits> if the exact same number is mentioned in the description of the example

pages/common/exo-compute-block-storage.md Outdated Show resolved Hide resolved
pages/common/exo-compute-block-storage.md Outdated Show resolved Hide resolved
pages/common/exo-compute-instance.md Outdated Show resolved Hide resolved
pages/common/exo-compute-instance.md Outdated Show resolved Hide resolved
pages/common/exo-compute-sks.md Outdated Show resolved Hide resolved
@davidlazch davidlazch force-pushed the davidlazch/exo-cli-pages branch from e58b0c7 to 4132f3f Compare May 20, 2025 08:39
@Managor
Copy link
Collaborator

Managor commented May 20, 2025

Please do not force push. It erases history and it's nice to see what changes were made after what comments.

Copy link
Collaborator

@Managor Managor left a comment

Choose a reason for hiding this comment

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

This seems to be in a good shape.

pages/common/exo-compute-instance.md Outdated Show resolved Hide resolved
pages/common/exo-compute-block-storage.md Outdated Show resolved Hide resolved
pages/common/exo-compute-sks.md Outdated Show resolved Hide resolved
pages/common/exo-compute.md Outdated Show resolved Hide resolved
pages/common/exo-dbaas.md Outdated Show resolved Hide resolved
pages/common/exo-iam.md Outdated Show resolved Hide resolved
pages/common/exo-storage.md Outdated Show resolved Hide resolved
pages/common/exo.md Outdated Show resolved Hide resolved
@davidlazch
Copy link
Contributor Author

Nice catch @MachiavelliII, we just did a Community revamp and this didn't cross my mind. Thanks!

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.

Thanks for your contribution, and welcome to tldr!

@Managor Managor dismissed sebastiaanspeck’s stale review May 28, 2025 08:53

Doesn't respond

@Managor
Copy link
Collaborator

Managor commented May 28, 2025

If you notice anything to improve, just fire another pull request.

@Managor Managor merged commit 95c9799 into tldr-pages:main May 28, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mass changes Changes that affect multiple pages. new command Issues requesting creation of a new page or PRs adding a new page for a command. review needed Prioritized PRs marked for reviews from maintainers.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants
Morty Proxy This is a proxified and sanitized view of the page, visit original site.