Skip to content

doc: clarify that any truthy value of shell is part of DEP0190#62249

Open
aduh95 wants to merge 2 commits intonodejs:mainfrom
aduh95:DEP0190-shell-truthy
Open

doc: clarify that any truthy value of shell is part of DEP0190#62249
aduh95 wants to merge 2 commits intonodejs:mainfrom
aduh95:DEP0190-shell-truthy

Conversation

@aduh95
Copy link
Contributor

@aduh95 aduh95 commented Mar 14, 2026

The current wording makes it sound like passing a string would not be a problem, remove the mention of true in the title and add an example with a string

@nodejs-github-bot
Copy link
Collaborator

Review requested:

  • @nodejs/userland-migrations

@nodejs-github-bot nodejs-github-bot added deprecations Issues and PRs related to deprecations. doc Issues and PRs related to the documentations. labels Mar 14, 2026
@aduh95 aduh95 added the dont-land-on-v20.x PRs that should not land on the v20.x-staging branch and should not be released in v20.x. label Mar 14, 2026
Co-authored-by: René <contact.9a5d6388@renegade334.me.uk>
@aduh95 aduh95 added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. commit-queue-squash Add this label to instruct the Commit Queue to squash all the PR commits into the first one. labels Mar 14, 2026
@aduh95 aduh95 added the commit-queue Add this label to land a pull request using GitHub Actions. label Mar 14, 2026
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. commit-queue Add this label to land a pull request using GitHub Actions. commit-queue-squash Add this label to instruct the Commit Queue to squash all the PR commits into the first one. deprecations Issues and PRs related to deprecations. doc Issues and PRs related to the documentations. dont-land-on-v20.x PRs that should not land on the v20.x-staging branch and should not be released in v20.x.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants