Closed Bug 1654859 Opened 4 years ago Closed 4 years ago

glean_parser: SUPERFLUOUS_NO_LINT isn't no_lint able ("The lintception")

Categories

(Data Platform and Tools :: Glean: SDK, defect, P1)

defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mdroettboom, Assigned: mdroettboom)

Details

Given the recent deletion{_|-}request debackle, it might be useful to ask our users to correct their metadata to deletion-request and add no_lint: MISSPELLED_PING as a workaround if they can't upgrade their version of Glean. However, this makes something that will warn (not error) on recent versions of glean_parser:

WARNING: SUPERFLUOUS_NO_LINT: legacy.ids.client_id: Superfluous no_lint entry 'MISSPELLED_PING'. Please remove it.

If we could also add SUPERFLUOUS_NO_LINT to no_lint, our users could write something that will be warning-free across glean_parser versions.

Priority: P3 → P1
Whiteboard: [telemetry:glean-rs:m?]
Assignee: nobody → mdroettboom
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.