You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Right now, the plugin treats all issues equally, which means we can't add issues that should be surfaced as warnings instead of errors.
My current thinking is to create a Severity enum that gets added as a required field to the Issue sealed type. Seeing as some best practices can have use-cases where they don't necessarily apply, and in the spirit of treating them as guidelines as opposed to strict rules, maybe that severity should have a default value of Warning, and issue authors can override that when contributing new ones.
The ConfigurableLogger class would also need to be made aware of this change, and maybe the surfaced warning messages would need to be slightly reworded to reflect this severity level and make users aware of the fact that they can add specific issues to their baseline given that they have appropriate reasons for breaking tradition at call sites where a guideline doesn't apply. Maybe this is a slippery slope though? I don't know.
The text was updated successfully, but these errors were encountered:
Right now, the plugin treats all issues equally, which means we can't add issues that should be surfaced as warnings instead of errors.
My current thinking is to create a
Severity
enum that gets added as a required field to theIssue
sealed type. Seeing as some best practices can have use-cases where they don't necessarily apply, and in the spirit of treating them as guidelines as opposed to strict rules, maybe that severity should have a default value ofWarning
, and issue authors can override that when contributing new ones.The
ConfigurableLogger
class would also need to be made aware of this change, and maybe the surfaced warning messages would need to be slightly reworded to reflect this severity level and make users aware of the fact that they can add specific issues to their baseline given that they have appropriate reasons for breaking tradition at call sites where a guideline doesn't apply. Maybe this is a slippery slope though? I don't know.The text was updated successfully, but these errors were encountered: