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
For clients of particular type only public method names matters. There is no need to suggest Async suffix for private methods. For example I have public methods which checks for arguments or states and throws exceptions immediately (these methods don't have 'async' keyword for 'fail-fast' reasons). And these methods delegate actual work to some private methods like "DoSomeActionInternal" or something like that. These method are hidden from outer client so it's pretty annoying to have suffix suggestions.
It would be nice to have some settings to turn off these suggestions for private methods.
The text was updated successfully, but these errors were encountered:
Could you please provide me with a test case? Like a sample class with some methods in it, and mark with comments where do you see a false positive warning.
For clients of particular type only public method names matters. There is no need to suggest Async suffix for private methods. For example I have public methods which checks for arguments or states and throws exceptions immediately (these methods don't have 'async' keyword for 'fail-fast' reasons). And these methods delegate actual work to some private methods like "DoSomeActionInternal" or something like that. These method are hidden from outer client so it's pretty annoying to have suffix suggestions.
It would be nice to have some settings to turn off these suggestions for private methods.
The text was updated successfully, but these errors were encountered: