Join GitHub today
GitHub is home to over 50 million developers working together to host and review code, manage projects, and build software together.
Sign upIntegrate snakecase-pkg into janitor #131
Conversation
|
Looks good. I have two questions:
Thanks Malte! |
|
Hi Sam,
Best, Malte |
|
Testcases for all available cases now included within |
|
Thanks for adding the additional test cases! I'll tinker with the description per your comment above, re-read the code, hopefully this week, and then merge it. |
|
Okay if I copy those tests to janitor? Seems like they should call |
|
Definitely ok. |
As mentioned in #96 and #130 For the latter I changed "%" into ".percent_" instead of
"_percent_", as commented in the code, since starting with "_" would have been converted to "x_percent" bymake.names().