Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork2.8k
Rules that are conditionally type-aware#6423
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
-
Something we've avoided in our project is rules that conditionally use type information. The reason we have avoided them historically them is because: There's currently just one rule in our project that uses conditional type info I do wonder if we're unnecessarily tying our hands with this restriction. Whilst it's true that most cases you just want the type-aware checks and that without type-information a rule will be buggy or incomplete, there are cases where some users might want to opt-in to additional checks in specific circumstances or type-aware fixers. Two quick examples off the top of my head:
Now that we're a more mature project, I think we're getting to the point where our users are well-versed enough to understand the nuances of this style. I'd be interested in us speccing this out a bit if people agree it's a good idea:
|
BetaWas this translation helpful?Give feedback.
All reactions
Replies: 2 comments 1 reply
-
I really would love to have a |
BetaWas this translation helpful?Give feedback.
All reactions
-
It's something I've been thinking about considering there's the upcoming Reworking the rule to set it up as a way to easily onboard onto the option would be really useful for the community. |
BetaWas this translation helpful?Give feedback.