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.

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

npm-unpublish: add page #14187

Merged
merged 6 commits into from
Oct 13, 2024
Merged

npm-unpublish: add page #14187

merged 6 commits into from
Oct 13, 2024

Conversation

Bucke200
Copy link
Contributor

@Bucke200 Bucke200 commented Oct 13, 2024

  • 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 command Issues requesting creation of a new page. label Oct 13, 2024
@tldr-bot
Copy link

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

pages/common/npm-unpublish.md:5 MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- Unpublish a specific package..."]
pages/common/npm-unpublish.md:5: TLDR007 Example descriptions should be surrounded by empty lines

Please fix the error(s) and push again.

@spageektti spageektti changed the title npm-unpublish: add new page npm-unpublish: add page Oct 13, 2024
Co-authored-by: Wiktor Perskawiec <[email protected]>
@Bucke200 Bucke200 marked this pull request as ready for review October 13, 2024 17:20
@Bucke200 Bucke200 requested a review from spageektti October 13, 2024 17:22
@sebastiaanspeck sebastiaanspeck mentioned this pull request Oct 13, 2024
67 tasks
pages/common/npm-unpublish.md Outdated Show resolved Hide resolved
@Bucke200 Bucke200 requested a review from spageektti October 13, 2024 17:45
@spageektti
Copy link
Member

LGTM, thanks for your contribution

@spageektti spageektti merged commit 1c4c70b into tldr-pages:main Oct 13, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new command Issues requesting creation of a new page.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants