Replace js-yaml with yaml
#1422
Conversation
Switch YAML parsing from `js-yaml` to `yaml`. Update the resolver to use `YAML.parse` when `JSON.parse` fails. Remove `js-yaml` and its types, add `yaml` as a runtime dependency. No public API changes. Signed-off-by: Sora Morimoto <sora@morimoto.io>
smorimoto
added
enhancement
🦋 Changeset detectedLatest commit: 8761a8f The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
![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 suggestions.
Reply with @codex fix comments to fix any unresolved comments.
About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you open a pull request for review, mark a draft as ready, or comment "@codex review". If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex fix this CI failure" or "@codex address that feedback".
|
@codex review |
smorimoto
force-pushed
the
js-yaml-to-yaml
branch
from
d3d342d to
8761a8f
Compare
|
Codex Review: Didn't find any major issues. What shall we delve into next? About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you open a pull request for review, mark a draft as ready, or comment "@codex review". If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex fix this CI failure" or "@codex address that feedback". |
Switch YAML parsing from
js-yamltoyaml. Update the resolver to useYAML.parsewhenJSON.parsefails. Removejs-yamland its types, addyamlas a runtime dependency. No public API changes.