New issue
Advanced search Search tips

Issue 923770 link

Starred by 1 user

Issue metadata

Status: Started
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

Make tast-lint support NOLINT for custom checks

Project Member Reported by derat@chromium.org, Jan 21 (2 days ago)

Issue description

I'm considering adding some stricter checks to tast-lint to make code reviews easier, but there will be a higher risk of false positives. It probably makes sense to support NOLINT comments so that issues can be suppressed.

It looks like tast-lint already supports NOLINT for ignoring golint warnings, but supporting it for our custom checks (which use ast.Walk) seems a bit trickier. I think I found an approach that works, though.
 

Sign in to add a comment