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

Normalizes topics to snake_case notation #890

Merged

Conversation

ilya-khadykin
Copy link
Contributor

This is meant to normalize topics to snake_case notation as discussed earlier.

Also relates to #888

Your feedback is welcomed

@tleen
Copy link
Member

tleen commented Aug 31, 2017

Lately it seems to be good practice to keep .txt documents compatible with markdown. As that was the case in the original file, I would recommend keeping the previous header style and limiting this PR to the snake casing.

This will also conflict with #888 (and potentially #886), should probably wait for #888 to be merged in, fix conflicts then this one would be good to go.

@ilya-khadykin
Copy link
Contributor Author

@tleen, got it, thanks

@tleen
Copy link
Member

tleen commented Sep 1, 2017

#888 is in 👍 snake_case that into this and we should be good to go!

@ilya-khadykin ilya-khadykin force-pushed the issue_887_convert_topics_to_snake_case branch from 5e589c5 to 64adcff Compare September 1, 2017 18:18
@ilya-khadykin
Copy link
Contributor Author

@tleen, PR has been updated accordingly

Copy link
Member

@tleen tleen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fantastic, thanks for getting this done!

@tleen tleen merged commit 59ebb95 into exercism:master Sep 5, 2017
@ilya-khadykin ilya-khadykin deleted the issue_887_convert_topics_to_snake_case branch September 5, 2017 16:18
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