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
I'm getting complaints of too many "false warnings" on the web+sql project I added this to. We could cover things like #123 and #245 by having a list of fully qualified type names that don't need disposal tracking in .editorconfig. Then have the defaults in an easily PR-able form.
Great project by the way!
The text was updated successfully, but these errors were encountered:
Is there any Attribute which could be used to accomplish that? (Turn off a given warning for a type). That would function similarly to the .editorconfig idea but would be more visible / more self-documenting.
I believe you're thinking of #126, which covers the attribute case for types defined in your own source code.
The problem I'm targeting here is to suppress warnings across the solution coming for disposable types from 3rd party libraries (including BCL) that actually don't need careful dispose tracking.
EditorConfig is already the standard for configuring analzyer warning levels and I think it could work nicely here. E.g. I'd make the analzyer warn by default, but compile error if I leak a SqlConnection, and not even show allegedly leaked DataColumns
I'm getting complaints of too many "false warnings" on the web+sql project I added this to. We could cover things like #123 and #245 by having a list of fully qualified type names that don't need disposal tracking in
.editorconfig
. Then have the defaults in an easily PR-able form.Great project by the way!
The text was updated successfully, but these errors were encountered: