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
3rd party or builtin interfaces may have methods requiring several arguments (e.g. Xamarin.Forms.IValueConverter, which defines Convert and ConvertBack with four arguments each). In these cases, CleanCode should not warn that the methods that implement the interfaces require too many arguments.
In general, I suggest that interface-implementing methods should be skipped entirely by the "too many arguments" check, and instead CleanCode should warn about this on the actual interface definitions.
The text was updated successfully, but these errors were encountered:
In general it would be nice to have an option to include "just my code" or to exclude the BCL / third party libraries. Fox example when implementing IList<T>.Add(T item) it raises a MethodNameNotMeaningful warning. Obviously there is nothing we can do about it except ignoring it.
3rd party or builtin interfaces may have methods requiring several arguments (e.g.
Xamarin.Forms.IValueConverter
, which definesConvert
andConvertBack
with four arguments each). In these cases, CleanCode should not warn that the methods that implement the interfaces require too many arguments.In general, I suggest that interface-implementing methods should be skipped entirely by the "too many arguments" check, and instead CleanCode should warn about this on the actual interface definitions.
The text was updated successfully, but these errors were encountered: