ampproject / ampproject/amppackager

Error out if TOML includes any unknown fields

Open
#130 0 comments 0 reactions 0 assignees View on GitHub
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.