Normalize module names #243
Conversation
| end | ||
|
|
||
| # helper classes | ||
|
|
There was a problem hiding this comment.
Nice - I like the Anvil -> Views + flattening and I think model makes more sense than vice.
In terms of other names:
I think polish is pretty clear on what it does - though I'm also not 100% convinced it should be separate functionality from Welds (ie, why not just have a polish or post_process hook on weld.
For Weld - Naming for this sort of functionality is hard. "Plugin" is already very overloaded. I think an alternative would be "Integration" or "Extension" or something like that, but its also not necessarily more descriptive than "Weld". "Weld" also feels some what similar to "Railtie", so I think its reasonable to keep it.
| @@ -3,16 +3,16 @@ | |||
| require 'rails/railtie' | |||
| require 'thor' | |||
|
|
|||
| require_relative 'smithy/tools' | |||
There was a problem hiding this comment.
I actually don't mind that utils are called tools. I felt it was pretty straightforward there.
We are not allowed to have fun anymore.