Warn on duplicate LLSD keys in autobuild package definitions#69
Conversation
|
CLA Assistant Lite bot: I have read the CLA Document and I hereby sign the CLA You can retrigger this bot by commenting recheck in this Pull Request |
|
@copilot Document new functions and any changes that would not be clear to a mid-level engineer or an engineer who is not already familiar with AutoBuild |
|
I have read the CLA Document and I hereby sign the CLA |
Addressed in d5126a6. I added docstrings for the duplicate-key helper functions and an inline note at the load site explaining why the warning has to run on raw LLSD XML before parsing collapses duplicate keys. |
|
The windows-latest test failure is unrelated to this branch. That's fixed in #71 |
Autobuild silently accepts duplicate LLSD map keys in package metadata, including repeated 3P/platform definitions introduced by merges. This change surfaces those conflicts at load time without changing current resolution behavior.
Behavior change
autobuild.xmland metadata files.Scope
llsd.parse()collapses duplicates into a single dict entry.Regression coverage
Example
This now warns on: