Remove deprecated safe prefix for js-yaml #176

Open

erikzaadi wants to merge 1 commit from erikzaadi/fix-js-yaml-upgrade into master

erikzaadi commented

2025-05-18 05:09:10 +00:00

(Migrated from github.com)

js-yaml have removed the safeDump and safeLoad (it's now safe by default).

This change prevents Error: Function yaml.safeLoad is removed in js-yaml 4. Use yaml.load instead, which is now safe by default. from happening

`js-yaml` have removed the `safeDump` and `safeLoad` (it's now safe by default). This change prevents `Error: Function yaml.safeLoad is removed in js-yaml 4. Use yaml.load instead, which is now safe by default.` from happening

👍 1

lukehinds commented

2025-07-10 21:00:50 +00:00

(Migrated from github.com)

any chance of merging this @jonschlinkert and cutting a release.?

🚀 8

This pull request has changes conflicting with the target branch.

lib/engines.js
package.json
test/parse-custom.js
test/parse-yaml.js

View command line instructions

Manual merge helper

Use this merge commit message when completing the merge manually.

Merge commit title

Merge pull request 'Remove deprecated safe prefix for js-yaml' (#176) from erikzaadi/fix-js-yaml-upgrade into master

Merge commit body

Reviewed-on: https://repo.reapertech.com/rtlabs-llm-agents/gms-gray-matter/pulls/176

Merge pull request 'Remove deprecated safe prefix for js-yaml' (#176) from erikzaadi/fix-js-yaml-upgrade into master

Reviewed-on: https://repo.reapertech.com/rtlabs-llm-agents/gms-gray-matter/pulls/176