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

Config Comment Cleanup #1803

Merged
merged 3 commits into from Feb 19, 2018
Merged

Conversation

phit
Copy link
Contributor

@phit phit commented Feb 16, 2018

Direct comparison of old and new config: https://gist.github.com/phit/589be51da2a2a024b0f9c3dff0c903f0/revisions


Mainly focused on comment formatting and rewriting things that weren't clear. Fixed a few typos here and there and generally tried to make everything more consistent.

Needs review and if anyone has any suggestions let me know, ill be happy to rewrite more and comment literally every setting if that's wanted.

Also includes setting deny-chunk-requests to false on Spongeforge, as per SpongePowered/SpongeForge#2003

This goes with SpongePowered/SpongeDocs#663, which I will update to these changes once this gets pulled.

Mainly focused on bringing comment formatting and rewriting things that
weren't clear. Fixed a few typos here and there and generally tried to
make everything more consistent.
@phit phit requested a review from Zidane February 16, 2018 13:18
@phit phit changed the title Config Cleanup Config Comment Cleanup Feb 16, 2018
@phit phit force-pushed the refactor/configs branch 2 times, most recently from 9b90e94 to 5a43cbe Compare February 16, 2018 13:54
true/false > enabled/disabled
'asynchronously'
Removed config for a feature that was removed mid 2016
SpongePowered@a1ac6cc#diff-4e6f4ff49ad98848b830a9c9b425940c
@bloodmc bloodmc merged commit 57702bf into SpongePowered:bleeding Feb 19, 2018
bloodmc pushed a commit that referenced this pull request Feb 19, 2018
* First pass of the big config cleanup

Mainly focused on bringing comment formatting and rewriting things that
weren't clear. Fixed a few typos here and there and generally tried to
make everything more consistent.

* Some suggestions by Inscrutable

true/false > enabled/disabled
'asynchronously'

* Some more cleanup

Removed config for a feature that was removed mid 2016
a1ac6cc#diff-4e6f4ff49ad98848b830a9c9b425940c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants