Repository navigation
chore(deps): bump actions/setup-node from 6 to 7 - #51
Conversation
Bumps [actions/setup-node](https://github.com/actions/setup-node) from 6 to 7. - [Release notes](https://github.com/actions/setup-node/releases) - [Commits](actions/setup-node@v6...v7) --- updated-dependencies: - dependency-name: actions/setup-node dependency-version: '7' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
LabelsThe following labels could not be found: Please fix the above issues or remove invalid values from |
Read metadata and evidence with a shared UTF-8 line iterator so a multiline rollout can exceed the JavaScript string limit without a whole-file allocation. Preserve parsing and file-error behavior. Refs #44 Signed-off-by: boulea7 <zln1905391059@163.com>
Avoid combining a representable line with following lines in a temporary string or rescanning the growing prefix on every chunk. Add a regression at the actual V8 string limit through the public metadata reader. Refs #44 Signed-off-by: boulea7 <zln1905391059@163.com>
fix: stream rollout JSONL instead of reading the whole file
|
Important Review skippedBot user detected. To trigger a single review, invoke the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Boulea7
left a comment
There was a problem hiding this comment.
Validated the setup-node v7 update against current main. The refreshed CI passes on Node 20/22/24 and the macOS/Windows smoke jobs.
Bumps actions/setup-node from 6 to 7.
Release notes
Sourced from actions/setup-node's releases.
... (truncated)
Commits
8207627Migrate to ESM and upgrade dependencies (#1574)04be95cAdd cache-primary-key and cache-matched-key as outputs (#1577)7c2c68ddocs: Update caching recommendations to mitigate cache poisoning risks (#1567)6a61c03Merge pull request #1569 from jasongin/update-actions-cache-5.1.030eb73bResolve high-severity audit issues4e1a87aUpdate dist360237fStrict equality4f8aac5Bump@actions/cacheto 5.1.0, log cache write deniedf4a67bbOnly usemirrorTokeningetManifestif it's provided (#1548)0355742Remove dummy NODE_AUTH_TOKEN export (#1558)Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)Summary by cubic
Upgrades
actions/setup-nodeto v7 in CI and release workflows, and streams rollout JSONL instead of reading the whole file.The rollout reader now processes JSONL line by line, so a rollout can exceed the JavaScript string limit without a whole-file allocation. Malformed lines are still skipped and file read errors still propagate.
Bug Fixes
Written for commit 79357ec. Summary will update on new commits.