Skip to content

bun-{remove,rm}: add page #19181

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

bun-{remove,rm}: add page #19181

wants to merge 5 commits into from
+32 −0

Conversation

@AttackThwarter
Copy link
Contributor

@AttackThwarter AttackThwarter commented Nov 3, 2025 •
edited
Loading

Checklist

  • The page(s) are in the correct platform directories: common, linux, osx, windows, sunos, android, etc.
  • 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 contains at most 5 new pages.
  • The PR is authored by me, or has been human-reviewed if it was created with AI or machine translation software.
  • The PR title conforms to the recommended templates.
  • Version of the command being documented (if known):

Reference issue: #19127

@github-actions github-actions bot added 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 Nov 3, 2025
dmmqz
dmmqz reviewed Nov 3, 2025
View reviewed changes
AttackThwarter and others added 2 commits November 3, 2025 23:16
@AttackThwarter @dmmqz
Update pages/common/bun-remove.md
e3330b2 
Co-authored-by: Dylan <145150333+dmmqz@users.noreply.github.com>
@AttackThwarter @dmmqz
Update pages/common/bun-remove.md
628ef9f 
Co-authored-by: Dylan <145150333+dmmqz@users.noreply.github.com>
@AttackThwarter AttackThwarter requested a review from dmmqz November 3, 2025 19:47
@Managor Managor mentioned this pull request Nov 4, 2025
Open
42 tasks
sebastiaanspeck
sebastiaanspeck reviewed Nov 4, 2025
View reviewed changes
@sebastiaanspeck sebastiaanspeck mentioned this pull request Nov 4, 2025
Open
7 tasks
AttackThwarter and others added 2 commits November 4, 2025 22:54
@AttackThwarter @sebastiaanspeck
Update pages/common/bun-remove.md
45bbf2c 
Co-authored-by: Sebastiaan Speck <12570668+sebastiaanspeck@users.noreply.github.com>
@AttackThwarter
style(bun-remove): apply header note
050e222
@AttackThwarter
Copy link
Contributor Author

AttackThwarter commented Nov 4, 2025 •
edited
Loading

Thanks @Managor for the clarification, and thanks @sebastiaanspeck for the suggestion!

I've now updated the PR to use the "header note" style for this page as a test.

I agree with your comments—it would be fantastic if this "beautiful disagreement" leads to a consensus and a definitive update in the style-guide.md. Having one clear, official standard for this would be super helpful for all contributors (especially new ones) in the future.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Managor Managor Managor left review comments

@sebastiaanspeck sebastiaanspeck sebastiaanspeck approved these changes

@dmmqz dmmqz dmmqz approved these changes

Assignees

No one assigned

Labels

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

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@AttackThwarter @sebastiaanspeck @Managor @dmmqz