gtristan commented on PR #1877:
URL: https://github.com/apache/buildstream/pull/1877#issuecomment-1763686005
If I understand correctly, this will improve correctness by raising an error
when bst loads redundant dict keys such as:
```yaml:
mydict:
foo: Pony
bar: Horsy
foo: Zebry
```
And the update to the test case is required because we're otherwise using
our yaml parsing to parse "variables" dictionaries of multiple elements which
would cause this error to be raised.
If my understanding is correct, then this looks good to me @abderrahim
please feel free to merge :)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
