2 ms·
Shopify has very strict rules to prevent that and they have been working to address that problem for YEARS. They have actually worked on Packwerk (which they us
by Fire-Dragon-DoL 5y ago
Shopify has very strict rules to prevent that and they have been working to address that problem for YEARS.
They have actually worked on Packwerk (which they use) which enforces the usage of constants when crossing boundaries, rather than any form of interface or duck typing.
The load time of a single test is a recurring problem in the Rails world though, so ActiveSupport adds up.
Monekypatching doesn't lead to crash (it can, but that's not the point), it leads to:
- Less extendable code
- Less learning
- Less maintainable code
Even if the damage is small, given how competitive is the world, why taking the chance for no benefit?
- joelbluminator 5y ago> Even if the damage is small, given how competitive is the world, why taking the chance for no benefit? Well that's the crux of it here, you see it as damage/no benefit while others in the Rails world disagree. It's a difference of opinions that won't be resolved tonight by arguing on Hacker News. I did like your comment about naming consistency in Ruby though, learned something new.