Skip to content

doc,lib,src,test: unflag sqlite module - #55890

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
cjihrig:unflag-sqlite
Nov 19, 2024
Merged

doc,lib,src,test: unflag sqlite module#55890
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
cjihrig:unflag-sqlite

Conversation

@cjihrig

@cjihrig cjihrig commented Nov 17, 2024

Copy link
Copy Markdown
Contributor

As requested in #55854...

This commit allows the node:sqlite module to be used without starting Node with a CLI flag. The module is still experimental.

Fixes: #55854

@nodejs-github-bot nodejs-github-bot added c++ Issues and PRs that require attention from people who are familiar with C++. needs-ci PRs that need a full CI run. process Issues and PRs related to the process subsystem. labels Nov 17, 2024
@codecov

codecov Bot commented Nov 17, 2024

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 88.41%. Comparing base (746b17e) to head (a54b0ac).
Report is 9 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #55890      +/-   ##
==========================================
- Coverage   88.41%   88.41%   -0.01%     
==========================================
  Files         654      654              
  Lines      188036   188036              
  Branches    36187    36182       -5     
==========================================
- Hits       166260   166257       -3     
- Misses      14989    14990       +1     
- Partials     6787     6789       +2     
Files with missing lines Coverage Δ
lib/internal/process/pre_execution.js 90.56% <100.00%> (+0.14%) ⬆️
src/node_options.cc 87.46% <ø> (ø)
src/node_options.h 98.27% <100.00%> (ø)

... and 38 files with indirect coverage changes

---- 🚨 Try these New Features:

@avivkeller avivkeller added the sqlite Issues and PRs related to the SQLite subsystem. label Nov 17, 2024
@cjihrig cjihrig added the request-ci Add this label to start a Jenkins CI on a PR. label Nov 17, 2024
@github-actions github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Nov 17, 2024
@nodejs-github-bot

nodejs-github-bot commented Nov 17, 2024

Copy link
Copy Markdown
Collaborator

@cjihrig cjihrig added 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. and removed needs-ci PRs that need a full CI run. labels Nov 18, 2024
This commit allows the node:sqlite module to be used without
starting Node with a CLI flag. The module is still experimental.

Fixes: nodejs#55854
@cjihrig

cjihrig commented Nov 18, 2024

Copy link
Copy Markdown
Contributor Author

Rebased to fix a minor conflict in the docs from #54181.

@cjihrig cjihrig added the request-ci Add this label to start a Jenkins CI on a PR. label Nov 18, 2024
@github-actions github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Nov 18, 2024
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@mcollina mcollina added the semver-minor PRs that contain new features and should be released in the next minor version. label Nov 18, 2024

@mcollina mcollina left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@marco-ippolito marco-ippolito added the notable-change PRs with changes that should be highlighted in changelogs. label Nov 18, 2024
@github-actions

Copy link
Copy Markdown
Contributor

The notable-change PRs with changes that should be highlighted in changelogs. label has been added by @marco-ippolito.

Please suggest a text for the release notes if you'd like to include a more detailed summary, then proceed to update the PR description with the text or a link to the notable change suggested text comment. Otherwise, the commit will be placed in the Other Notable Changes section.

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Nov 19, 2024
@nodejs-github-bot
nodejs-github-bot merged commit 1d01ad6 into nodejs:main Nov 19, 2024
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Landed in 1d01ad6

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. c++ Issues and PRs that require attention from people who are familiar with C++. notable-change PRs with changes that should be highlighted in changelogs. process Issues and PRs related to the process subsystem. semver-minor PRs that contain new features and should be released in the next minor version. sqlite Issues and PRs related to the SQLite subsystem.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Unflag node:sqlite

8 participants