Repository navigation
chore(deps): Bump the cargo-major group across 1 directory with 3 updates - #2510
dependabot[bot] wants to merge 1 commit into
Conversation
dependabot
Bot
added
dependencies
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: a286e0032a
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| napi-derive = { version = "3.6.2" } | ||
| p256 = { version = "0.14", features = ["ecdsa"] } | ||
| pg_query = { version = "=6.2.1", default-features = false } | ||
| pg_query = { version = "=18.1.0", default-features = false } |
There was a problem hiding this comment.
The root Cargo.lock still resolves pg_query 6.2.1, miniz_oxide 0.8.5, and saphyr-parser 0.1.0, while this commit changes their exact requirements and refreshes only the nested WASM SDK lockfile. Cargo defines --locked as asserting that Cargo.lock remains unchanged, so the root workspace's documented locked check and test commands cannot resolve this manifest and fail before compiling; regenerate and commit the root lockfile as part of the bump.
AGENTS.md reference: AGENTS.md:L292-L300
Useful? React with 👍 / 👎.
| napi-derive = { version = "3.6.2" } | ||
| p256 = { version = "0.14", features = ["ecdsa"] } | ||
| pg_query = { version = "=6.2.1", default-features = false } | ||
| pg_query = { version = "=18.1.0", default-features = false } |
There was a problem hiding this comment.
Once the lockfile/build issues are corrected, this switches offline SQL validation to PostgreSQL 18.6 grammar even though BReg still supports PostgreSQL 17. validate_alter_table accepts AtAddConstraint without inspecting the new constraint options, so a reviewed step such as ALTER TABLE ... ADD CONSTRAINT ... CHECK (...) NOT ENFORCED can pass tooling validation and then fail with a syntax error during activation on a supported PostgreSQL 17 deployment. Pin validation to PostgreSQL 17 grammar or explicitly reject PostgreSQL 18-only options and add a PostgreSQL 17 migration test before taking this bump.
AGENTS.md reference: products/breg/AGENTS.md:L200-L208
Useful? React with 👍 / 👎.
| rustls-native-certs = { version = "0.8.4" } | ||
| ryu-js = { version = "=1.0.3" } | ||
| saphyr-parser = { version = "=0.1.0" } | ||
| saphyr-parser = { version = "=0.3.3" } |
There was a problem hiding this comment.
The maintained crates/registry-platform-yaml/README.md:427-439 still says this dependency is pinned exactly to 0.1.0 and presents version-specific unsafe-code, source-size, dependency, event-suite, and cargo deny evidence for that release. Moving the parser trust boundary to 0.3.3 without updating this section leaves the current dependency described by evidence gathered for different source and makes the documented pin false; re-vet 0.3.3 and update the parser-choice evidence alongside the bump.
AGENTS.md reference: products/platform/AGENTS.md:L14-L17
Useful? React with 👍 / 👎.
5d3b442 to
d736f6c
Compare
…ates Bumps the cargo-major group with 3 updates in the /products/breg/wasm-handler-sdk directory: [miniz_oxide](https://github.com/Frommi/miniz_oxide), [pg_query](https://github.com/pganalyze/pg_query.rs) and [saphyr-parser](https://github.com/saphyr-rs/saphyr). Updates `miniz_oxide` from 0.8.5 to 0.9.1 - [Changelog](https://github.com/Frommi/miniz_oxide/blob/master/CHANGELOG.md) - [Commits](https://github.com/Frommi/miniz_oxide/commits) Updates `pg_query` from 6.2.1 to 18.1.0 - [Changelog](https://github.com/pganalyze/pg_query.rs/blob/main/CHANGELOG.md) - [Commits](pganalyze/pg_query.rs@v6.2.1...v18.1.0) Updates `saphyr-parser` from 0.1.0 to 0.3.3 - [Release notes](https://github.com/saphyr-rs/saphyr/releases) - [Commits](saphyr-rs/saphyr@v0.1.0...v0.3.3) --- updated-dependencies: - dependency-name: miniz_oxide dependency-version: 0.9.1 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: cargo-major - dependency-name: pg_query dependency-version: 18.1.0 dependency-type: direct:production update-type: version-update:semver-major dependency-group: cargo-major - dependency-name: saphyr-parser dependency-version: 0.3.3 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: cargo-major ... Signed-off-by: dependabot[bot] <support@github.com>
d736f6c to
e766859
Compare
|
Looks like these dependencies are updatable in another way, so this is no longer needed. |
Bumps the cargo-major group with 3 updates in the /products/breg/wasm-handler-sdk directory: miniz_oxide, pg_query and saphyr-parser.
Updates
miniz_oxidefrom 0.8.5 to 0.9.1Changelog
Sourced from miniz_oxide's changelog.
... (truncated)
Commits
Updates
pg_queryfrom 6.2.1 to 18.1.0Changelog
Sourced from pg_query's changelog.
... (truncated)
Commits
20f3a18Release 18.1.0 (#79)Updates
saphyr-parserfrom 0.1.0 to 0.3.3Commits
9a60beasaphyr v0.3.3eac3d1dCHANGELOG; update v0.3.3 release notes draft5d16228Merge pull request #147 from wleeaf/fix/emitter-literal-cr-bomabe2bafgarden: add remote for@wleeaf58916befix(emitter): indent root literal blocks with document marker linesaf9a829fix(emitter): keep trailing empty lines of multiline strings with |+d629589fix(emitter): don't write carriage returns or a BOM in literal blockseb0feb1cargo: update clipboard-win to non-yanked version31fd871saphyr v0.3.22a3e03dMerge pull request #145 from spicysharkco/fix/emitter-quote-bom