-
Notifications
You must be signed in to change notification settings - Fork 198
Add case insensitive option to export name rule #430
Comments
Huh, I would think the rule should only be case insensitive 😕. +1. Otherwise how would it work when a repo initialized on a case sensitive file system, then ported to a file system that is case insensitive? We probably should not get in the game of encouraging OS-specific patterns.
+1 to an option added to give it a whitelist of characters it can safely ignore. In your case, that looks like -1 to the pluralization logic in tableize. Pluralizing words is a difficult task and generally relies on having a package with much specialized logic large lists of special cases. Let's not take a dependency on that. |
How would you like this option to work with the existing args that the rule takes?
Change it to allow something like this? {
"export-name": [true, {
"allow": ["myChartOptions"],
"ignore-case": true
}]
} |
@roblourens that looks like a good path forward! If the "legacy" form of |
I would like to still have this rule enabled, but I want the code style to be the same across our project.
It would be nice if there was an option to support tableize too. So the filename can be my_class.ts and the class name MyClass.
The text was updated successfully, but these errors were encountered: