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

doc: update --max-semi-space-size description in cli.md #55495

Merged
merged 1 commit into from
Nov 5, 2024

Conversation

joebowbeer
Copy link
Contributor

Fixes #55487

@nodejs-github-bot nodejs-github-bot added cli Issues and PRs related to the Node.js command line interface. doc Issues and PRs related to the documentations. labels Oct 22, 2024
doc/api/cli.md Outdated Show resolved Hide resolved
@RedYetiDev RedYetiDev added the dont-land-on-v18.x PRs that should not land on the v18.x-staging branch and should not be released in v18.x. label Oct 23, 2024
@RedYetiDev
Copy link
Member

RedYetiDev commented Oct 23, 2024

Please amend your commit message to follow the guidelines of commits in this repo. Something similar to your PR title will work.

Congrats on your first contribution 🎉

@joebowbeer
Copy link
Contributor Author

@RedYetiDev I addressed both of your comments.

@joebowbeer
Copy link
Contributor Author

Should I lowercase the commit title?

@lpinca
Copy link
Member

lpinca commented Oct 23, 2024

Yes, Update -> update.

@joebowbeer
Copy link
Contributor Author

joebowbeer commented Oct 23, 2024

@lpinca @RedYetiDev PTAL

@joebowbeer joebowbeer changed the title doc: Update --max-semi-space-size description in cli.md doc: update --max-semi-space-size description in cli.md Oct 23, 2024
@RedYetiDev RedYetiDev added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Oct 24, 2024
@joebowbeer
Copy link
Contributor Author

joebowbeer commented Nov 5, 2024

@aduh95 PTAL

Is there anything more needed before merging?

@aduh95 aduh95 merged commit 25f8474 into nodejs:main Nov 5, 2024
17 checks passed
@aduh95
Copy link
Contributor

aduh95 commented Nov 5, 2024

Landed in 25f8474

@joebowbeer joebowbeer deleted the patch-1 branch November 5, 2024 23:17
aduh95 pushed a commit that referenced this pull request Nov 6, 2024
PR-URL: #55495
Fixes: #55487
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
aduh95 pushed a commit that referenced this pull request Nov 6, 2024
PR-URL: #55495
Fixes: #55487
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
tpoisseau pushed a commit to tpoisseau/node that referenced this pull request Nov 21, 2024
PR-URL: nodejs#55495
Fixes: nodejs#55487
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
author ready PRs that have at least one approval, no pending requests for changes, and a CI started. cli Issues and PRs related to the Node.js command line interface. doc Issues and PRs related to the documentations. dont-land-on-v18.x PRs that should not land on the v18.x-staging branch and should not be released in v18.x.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

--max-semi-space-size doc is wrong after Node 18
6 participants