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

Running clconf on overlapping YAML files does not result in proper merge of boolean properties. #20

Closed
cjvirtucio87 opened this issue Jul 23, 2020 · 3 comments

Comments

@cjvirtucio87
Copy link

Problem

Running clconf on overlapping YAML files does not result in proper merge of boolean properties.

Steps to Reproduce

Create the following YAML files:

# foo.yml
---
foo: true

# bar.yml
---
foo: false

Run clconf with those files:

clconf --ignore-env --yaml foo.yml --yaml bar.yml

Expected Outcome

foo: false

Actual Outcome

foo: true

Workarounds

N/A

Resolution

TBD

@lucastheisen
Copy link
Contributor

This is an upstream behavior (reported by issue 129 and issue 89). There seems to be an option to change this behavior. Will have to try it out and see effect it has on existing test cases.

@TomOperator
Copy link

For those of you who need it, @Pothulapati built a fork that solves this here.

@lucastheisen
Copy link
Contributor

lucastheisen commented Aug 23, 2020

Fixed by #22, released as v2.0.16

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

No branches or pull requests

3 participants