Skip to content
This repository has been archived by the owner on Jun 20, 2023. It is now read-only.

jsonnetfmt -i kube*.libsonnet #68

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

davidquarles
Copy link

I've used this library extensively at my current and previous gigs and can't thank you all enough for the work! We currently have our own private fork with a handful of additions that I intend to submit here but am as of now just trying to reconcile the cosmetic changes introduced by a basic invocation of jsonnetfmt / the vim-jsonnet plugin, so that we can more easily see the forest through the trees. This is, for the most part, just a switch from double quotes to single quotes (the default).

Is this desirable? I've only touched the files that we actually rely on downstream to minimize the footprint of this PR, but I can easily loop in all of *.*sonnet, too.

Thanks!

@jjo
Copy link
Contributor

jjo commented Oct 6, 2021

As always it's a matter of style :), also enforced in tests/Makefile via JSONNET_FMT settings there.
Agree that we should switch to jsonnetfmt stock style, note that indeed tests will need to be fixed.

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

Successfully merging this pull request may close these issues.

2 participants