Skip to content

doc: clarify sqlite bare named parameter default - #64919

Closed
sanzofr wants to merge 1 commit into
nodejs:mainfrom
sanzofr:doc-sqlite-bare-params
Closed

doc: clarify sqlite bare named parameter default#64919
sanzofr wants to merge 1 commit into
nodejs:mainfrom
sanzofr:doc-sqlite-bare-params

Conversation

@sanzofr

Copy link
Copy Markdown

Fixes: #61823

Clarify that allowBareNamedParameters defaults to true, so bare named
parameters are allowed by default. The previous wording incorrectly implied
that the prefix character is required unless bare named parameters are
explicitly enabled.

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/sqlite

@nodejs-github-botnodejs-github-bot added doc Issues and PRs related to the documentations. sqlite Issues and PRs related to the SQLite subsystem. labels Aug 1, 2026
@trivikr

Copy link
Copy Markdown
Member

@sanzofr Can you please sign the commit and force push?

It's required as per instructions

Your commit must contain the Signed-off-by line with your name and email address as an acknowledgement that you agree to the Developer Certificate of Origin.

Signed-off-by: Sanzo <sanzofr62@gmail.com>
@sanzofr
sanzofrforce-pushed the doc-sqlite-bare-params branch from be89f7a to e09aacdCompareAugust 7, 2026 05:59
@sanzofr

Copy link
Copy Markdown
Author

Done. i've signed off the commit and force-pushed the changes.

@trivikr

trivikr commented Aug 8, 2026

Copy link
Copy Markdown
Member

I just noticed that there's an existing PR which was posted before this #62009, and was linked in the issue.

@trivikrtrivikr added the duplicate Issues and PRs that are duplicates of other issues or PRs. label Aug 8, 2026
@trivikrtrivikr closed this Aug 8, 2026
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docIssues and PRs related to the documentations.duplicateIssues and PRs that are duplicates of other issues or PRs.sqliteIssues and PRs related to the SQLite subsystem.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

docs: sqlite statement.setAllowBareNamedParameters()

3 participants

@sanzofr@nodejs-github-bot@trivikr