Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for merge keys #1

Open
1 of 3 tasks
Timothy-Gonzalez opened this issue Jun 18, 2022 · 1 comment
Open
1 of 3 tasks

Add support for merge keys #1

Timothy-Gonzalez opened this issue Jun 18, 2022 · 1 comment
Assignees
Labels
enhancement New feature or request

Comments

@Timothy-Gonzalez
Copy link
Owner

Timothy-Gonzalez commented Jun 18, 2022

YAML merge keys, as defined here should be supported.

While not part of the standard specification, it is still recommended to support as:

The use of these tags is not mandatory. However these tags represent types that are useful across a wide range of applications and it is strongly recommended they be used whenever appropriate to promote interoperability.
Source

This will technically be a breaking change.

Example Behavior

a: &a #Anchor
  b: 1
c: *a #Normal alias
d:
  <<: *a #Merge alias
  e: 2

Would result in:

a:
  b: 1
c:
  b: 1
d:
  b: 1
  e: 2

Tasks

  • Find crate which supports yaml merge keys and integratable support for serde_yaml
  • Implement merge keys
  • Create merge_key test
@Timothy-Gonzalez Timothy-Gonzalez added the enhancement New feature or request label Jun 18, 2022
@Timothy-Gonzalez Timothy-Gonzalez self-assigned this Jun 18, 2022
@Timothy-Gonzalez
Copy link
Owner Author

Possible canidate: yaml_merge_keys. It even has built-in support for serde_yaml through the serde_yaml feature.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant