add topics to grains - #954
Conversation
ilya-khadykin
left a comment
There was a problem hiding this comment.
Thanks for the PR
Please read my comment
| "topics": [ | ||
|
|
||
| "bitwise-operations", | ||
| "control-flow-(if-else-statements)", |
There was a problem hiding this comment.
We've switched to snake_case naming convention for topics to increase consistency among tracks, so could you please choose them from the following list - https://github.com/exercism/problem-specifications/blob/master/TOPICS.txt
Please follow the convention
There was a problem hiding this comment.
Sorry for the delay, I had switched it to snake_case and removed topic that is not presented in the TOPICS.txt.
|
Thanks, @Zaccc123 It's a best practice to squash your commits https://github.com/exercism/docs/blob/master/contributing/git-basics.md#squashing. It isn't a big deal since I can do it myself, but it would be awesome to just click on 'Merge pull request' button 😃 |
fixes #830