ampproject / ampproject/amppackager
Error out if TOML includes any unknown fields
Open
fixit
- Dominant language
- Go
- Stars
- 141
- Forks
- 46
- PR merge metrics
- No merged PRs in 30d
Description
Currently, the toml parser just silently ignores any fields that don't match the struct. This means that typos are hard to diagnose, and backwards-incompatible changes to the config will require special care to notify existing users.
Contributor guide
Research direction
Locate the TOML parser entry point and the configuration structs it populates. Add coverage for unknown fields, including the expected error, and confirm that valid configuration fields continue to parse successfully.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- go
- Domain
- tooling
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100