Skip to content

Normalize the config file to minimize noise later#970

Merged
stkent merged 1 commit into
masterfrom
normalize-configs
Oct 23, 2017
Merged

Normalize the config file to minimize noise later#970
stkent merged 1 commit into
masterfrom
normalize-configs

Conversation

@kytrinyx
Copy link
Copy Markdown
Member

We will be scripting a number of changes to the track configuration.
This runs configlet fmt, which ensures that any further changes can be minimized.

See exercism/meta#95

We will be scripting a number of changes to the track configuration.
This runs configlet fmt, which ensures that any further changes can be minimized.
@stkent
Copy link
Copy Markdown
Contributor

stkent commented Oct 23, 2017

Merging, though please note we have a number of "add topics" issues pending that are likely to (re-)introduce inconsistencies (as you know, per exercism/configlet#78). Until there's a way to check whether format is up-to-date per fmt in CI, it might be safest to execute fmt as part of the scripted change itself.

bitmoji

@stkent stkent merged commit dfd0b63 into master Oct 23, 2017
@stkent stkent deleted the normalize-configs branch October 23, 2017 08:59
@kytrinyx
Copy link
Copy Markdown
Member Author

There's some discussion in exercism/meta#95 about how this normalization degrades the readability of the configs.
Unfortunately, this means that I'm going to have to submit yet another normalization once that discussion concludes and we update the serializer.

I'm sorry about the noise, and any extra work that this is causing you!

Meanwhile, if you have any opinions on the topic, I'd appreciate it if you added your thoughts in exercism/meta#95

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

Successfully merging this pull request may close these issues.

2 participants