build(deps): bump js-yaml from 4.2.0 to 5.2.0#635
Conversation
|
Bumps [js-yaml](https://github.com/nodeca/js-yaml) from 4.2.0 to 5.2.0. - [Changelog](https://github.com/nodeca/js-yaml/blob/master/CHANGELOG.md) - [Commits](nodeca/js-yaml@4.2.0...5.2.0) --- updated-dependencies: - dependency-name: js-yaml dependency-version: 5.2.0 dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
2f8900d to
7401bd2
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #635 +/- ##
=======================================
Coverage 99.86% 99.86%
=======================================
Files 7 7
Lines 736 736
=======================================
Hits 735 735
Misses 1 1 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
js-yaml v5 removes the default export and ships its own types.
Switch src/content.js to named imports ({ JSON_SCHEMA, load }) and
drop the now-redundant @types/js-yaml dev dependency.
Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>
6b4610d to
db8217b
Compare
zimeg
left a comment
There was a problem hiding this comment.
@dependabot LGTM! A few migration changes but nothing so breaking:
🔗 https://github.com/nodeca/js-yaml/blob/master/docs/migrate_v4_to_v5.md
| "@biomejs/biome": "^2.5.1", | ||
| "@changesets/cli": "^2.31.0", | ||
| "@types/flat": "^5.0.5", | ||
| "@types/js-yaml": "^4.0.9", |
There was a problem hiding this comment.
🫡 praise: Farewell separate @types/js-yaml package and thank you!
js-yaml 5.x adopts strict YAML 1.2 flow-scalar indentation and throws `deficient indentation` on a double-quoted value that wraps across a blank line onto a column-0 continuation line. A `payload: |` block strips its common indent, so keys and wrapped values land at column 0 -- the exact shape of a multi-paragraph Slack message. Payloads that worked in v3.0.3 broke in v3.0.4 (#635). 5.x offers no option or schema to relax this (CORE_SCHEMA, YAML11_SCHEMA and the default all throw identically), so revert js-yaml to ^4.2.0 and restore the @types/js-yaml devDependency. This makes the regression test added in the previous commit pass. Fixes #637 Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>
Bumps js-yaml from 4.2.0 to 5.2.0.
Changelog
Sourced from js-yaml's changelog.
... (truncated)
Commits
c28ed5e5.2.0 released125cd5aAddmaxAliasesoption3105455ReplacemaxMergeSeqLengthoption withmaxTotalMergeKeys(more robust)39d00d6numbers: Drop boxed numbers support, simplify .identify() checks, clarify rou...eb5cb5bfix: round-trip integers that stringify in exponential notation (#771)89024c4Update migration info, close #770f1e45cd5.1.0 released53b22beFix constructor coveragea1eaa2bFix quote style options and restore forceQuotes0532e7dAdd finalizers for immutable collection tags