-
-
Notifications
You must be signed in to change notification settings - Fork 62
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 Toml support #889
Add Toml support #889
Conversation
Signed-off-by: Olblak <me@olblak.com>
Signed-off-by: Olblak <me@olblak.com>
I need to test if Dasel preserve Toml comments |
Well it doesn't |
Signed-off-by: Olblak <me@olblak.com>
Signed-off-by: Olblak <me@olblak.com>
Signed-off-by: Olblak <me@olblak.com>
Signed-off-by: Olblak <me@olblak.com>
Signed-off-by: Olblak <me@olblak.com>
Signed-off-by: Olblak <me@olblak.com>
Signed-off-by: Olblak <me@olblak.com>
Signed-off-by: Olblak <me@olblak.com>
I played a bit with the multiple update which is super powerfull. I already works for target, I'll add support for condition but I don't think of a usage for source |
Signed-off-by: Olblak <me@olblak.com>
Signed-off-by: Olblak <me@olblak.com>
Signed-off-by: Olblak <me@olblak.com>
Well after additional thought, I think I need a new boolean to enable/disable the multiple query because in the case of the following query, we can't guess automatically the meaning Should
I am considering adding a spec parameter named "multipe" to enable multiple query, with default set to false |
Signed-off-by: Olblak <me@olblak.com>
So I decided to add the flag |
Signed-off-by: Olblak <me@olblak.com>
Signed-off-by: Olblak me@olblak.com
Add support to toml for source/condition/target
#870
Fix #XXX
Test
To test this pull request, you can run the following commands:
cp pkg/plugins/resources/toml go test
Additional Information
Tradeoff
Same approach than #885
I am leveraging Dasel
Potential improvement