Skip to content

build: fix typo 'Stoage' to 'Storage' in help text - #58777

Merged
nodejs-github-bot merged 2 commits into
nodejs:mainfrom
ganglike248:main
Jun 23, 2025
Merged

build: fix typo 'Stoage' to 'Storage' in help text#58777
nodejs-github-bot merged 2 commits into
nodejs:mainfrom
ganglike248:main

Conversation

@ganglike248

Copy link
Copy Markdown
Contributor

Fix spelling error in --without-sqlite option help message.

This PR fixes a typo in the configure script where 'Stoage' was incorrectly written instead of 'Storage' in the help text for the --without-sqlite option on line 871.

Changes:

  • Fixed typo in configure line 871: 'Stoage' → 'Storage'
  • Corrected help message: "build without SQLite (disables SQLite and Web Storage API)"

Why this change is needed:
The typo could confuse developers reading the help text for build configuration options. The correct term is "Web Storage API", not "Web Stoage API".

Testing:

  • Verified the configure script runs without errors
  • Confirmed help text displays correctly with python configure --help

This is a minor documentation fix that doesn't require tests or changelog entry.

Additional Note:
This is my first contribution to this project, but I wanted to make a small contribution by fixing this typo to help improve the documentation quality.


Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
have the right to submit it under the open source license
indicated in the file; or

(b) The contribution is based upon previous work that, to the best
of my knowledge, is covered under an appropriate open source
license and I have the right under that license to submit that
work with modifications, whether created in whole or in part
by me, under the same open source license (unless I am
permitted to submit under a different license), as indicated
in the file; or

(c) The contribution was provided directly to me by some other
person who certified (a), (b) or (c) and I have not modified
it.

(d) I understand and agree that this project and the contribution
are public and that a record of the contribution (including all
personal information I submit with it, including my sign-off) is
maintained indefinitely and may be redistributed consistent with
this project or the open source license(s) involved.

@nodejs-github-botnodejs-github-bot added build Issues and PRs related to build files or the CI. needs-ci PRs that need a full CI run. labels Jun 21, 2025
@ganglike248ganglike248 changed the title configure: fix typo 'Stoage' to 'Storage' in help textbuild: fix typo 'Stoage' to 'Storage' in help textJun 21, 2025
@lpincalpinca added request-ci Add this label to start a Jenkins CI on a PR. commit-queue Add this label to land a pull request using GitHub Actions. labels Jun 22, 2025
@github-actionsgithub-actionsBot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jun 22, 2025
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@ganglike248

Copy link
Copy Markdown
ContributorAuthor

Hello maintainers,

Could you please approve the workflow runs? I need to retrigger the CI due to test failures that appear to be unrelated to my changes. 🥹

Thank you for your time and consideration. 🙏

@ganglike248
ganglike248 requested a review from lpincaJune 22, 2025 07:41
@lpincalpinca added commit-queue-squash Add this label to instruct the Commit Queue to squash all the PR commits into the first one. request-ci Add this label to start a Jenkins CI on a PR. labels Jun 22, 2025
@github-actionsgithub-actionsBot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jun 22, 2025
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

CI: https://ci.nodejs.org/job/node-test-pull-request/67609/

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-botnodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Jun 23, 2025
@nodejs-github-bot
nodejs-github-bot merged commit 32be16b into nodejs:mainJun 23, 2025
42 checks passed
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Landed in 32be16b

RafaelGSS pushed a commit that referenced this pull request Jun 24, 2025
PR-URL: #58777
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>
Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com>
Reviewed-By: Ethan Arrowood <ethan@arrowood.dev>
aduh95 pushed a commit that referenced this pull request Jul 21, 2025
PR-URL: #58777
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>
Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com>
Reviewed-By: Ethan Arrowood <ethan@arrowood.dev>
aduh95 pushed a commit that referenced this pull request Jul 24, 2025
PR-URL: #58777
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>
Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com>
Reviewed-By: Ethan Arrowood <ethan@arrowood.dev>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

buildIssues and PRs related to build files or the CI.commit-queue-squashAdd this label to instruct the Commit Queue to squash all the PR commits into the first one.needs-ciPRs that need a full CI run.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants

@ganglike248@nodejs-github-bot@jasnell@lpinca@Ethan-Arrowood@juanarbol@marco-ippolito