armake icon indicating copy to clipboard operation
armake copied to clipboard

Add line comments to specifically ignore a issue on a line

Open CreepPork opened this issue 6 years ago • 4 comments

Similarly like ReSharper does it. Could be useful to mitigate some issues that are only picked up by armake and don't affect actual application performance.

For example:

// ReSharper disable once ReplaceWithStringIsNullOrEmpty

CreepPork avatar Oct 14 '18 18:10 CreepPork

what kind of issues do you mean? Armake is not a linter or syntax checker. If it stumbles upon an error, it's an error.

dedmen avatar Nov 20 '18 12:11 dedmen

@dedmen See #84.

CreepPork avatar Nov 20 '18 14:11 CreepPork

That was a bug in the armake parser. Bugs should be fixed rather than marked with comments to ignore the resulting warnings.

Only real thing that you could mark to ignore is the unquoted strings stuff. But that is really a error in config and should be fixed accordingly.

dedmen avatar Nov 20 '18 14:11 dedmen

It’s up for debate. I’d agree with issues being fixed not ignored but sometimes you could run into a wall and this could be a temp fix.

CreepPork avatar Nov 20 '18 14:11 CreepPork