Issue #87 and PR #89 implement the switch from gopkg.in/yaml.v3 to github.com/goccy/go-yaml as originally required by #57. However, the CONVENTIONS.md approved dependency allowlist still lists only gopkg.in/yaml.v3.
Per the documented process in CONVENTIONS.md:
Open a PR that ONLY updates this table
Requires explicit approval from Aaron
After merge, a separate PR may use the package
Action Required
Update the approved third-party packages table in CONVENTIONS.md:
This unblocks PR #89 which implements the actual migration.
Acceptance Criteria
CONVENTIONS.md table updated with github.com/goccy/go-yaml
scripts/check-deps.sh passes with the new allowlist
Aaron approves the dependency change
## Context
Issue #87 and PR #89 implement the switch from `gopkg.in/yaml.v3` to `github.com/goccy/go-yaml` as originally required by #57. However, the CONVENTIONS.md approved dependency allowlist still lists only `gopkg.in/yaml.v3`.
Per the documented process in CONVENTIONS.md:
1. Open a PR that ONLY updates this table
2. Requires explicit approval from Aaron
3. After merge, a separate PR may use the package
## Action Required
Update the approved third-party packages table in CONVENTIONS.md:
- Remove: `gopkg.in/yaml.v3`
- Add: `github.com/goccy/go-yaml` (and subpackages `ast`, `parser`)
This unblocks PR #89 which implements the actual migration.
## Acceptance Criteria
- [ ] CONVENTIONS.md table updated with `github.com/goccy/go-yaml`
- [ ] `scripts/check-deps.sh` passes with the new allowlist
- [ ] Aaron approves the dependency change
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Context
Issue #87 and PR #89 implement the switch from
gopkg.in/yaml.v3togithub.com/goccy/go-yamlas originally required by #57. However, the CONVENTIONS.md approved dependency allowlist still lists onlygopkg.in/yaml.v3.Per the documented process in CONVENTIONS.md:
Action Required
Update the approved third-party packages table in CONVENTIONS.md:
gopkg.in/yaml.v3github.com/goccy/go-yaml(and subpackagesast,parser)This unblocks PR #89 which implements the actual migration.
Acceptance Criteria
github.com/goccy/go-yamlscripts/check-deps.shpasses with the new allowlist