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

Normalize the config file to minimize noise later #193

Merged
merged 1 commit into from
Oct 23, 2017
Merged

Conversation

kytrinyx
Copy link
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.
@yurrriq yurrriq merged commit e4305ff into master Oct 23, 2017
@kytrinyx
Copy link
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

@haus haus deleted the normalize-configs branch December 12, 2017 05:43
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