Skip to content

chore(lint): remove obsolete spec exceptions#29601

Open
caugner wants to merge 3 commits intomainfrom
remove-obsolete-spec-url-exceptions
Open

chore(lint): remove obsolete spec exceptions#29601
caugner wants to merge 3 commits intomainfrom
remove-obsolete-spec-url-exceptions

Conversation

@caugner
Copy link
Copy Markdown
Contributor

@caugner caugner commented May 4, 2026

Summary

Updates the spec_url exception list, removing two no longer used entries.

Also extracts this exception list to a lint/common/spec-url-exceptions.txt, without requiring approval from @mdn/engineering.

Test results and supporting details

Checked that the other ones are all still in use, and not in web-specs.

Related issues

Triggered by #29582 (comment).

@caugner caugner requested a review from a team as a code owner May 4, 2026 16:17
@caugner caugner requested a review from LeoMcA May 4, 2026 16:17
@github-actions github-actions Bot added infra Infrastructure issues (npm, GitHub Actions, releases) of this project linter Issues or pull requests regarding the tests / linter of the JSON files. size:s [PR only] 7-24 LoC changed labels May 4, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 4, 2026

Tip: Review these changes grouped by change (recommended for most PRs), or grouped by feature (for large PRs).

Exclude it in CODEOWNERS file.
@github-actions github-actions Bot added size:l [PR only] 101-1000 LoC changed and removed size:s [PR only] 7-24 LoC changed labels May 4, 2026
@caugner
Copy link
Copy Markdown
Contributor Author

caugner commented May 4, 2026

I realize updates to spec url exceptions shouldn't require approval from @mdn/engineering, so I pushed 02c74a5 to extract it to a text file, and exclude it from CODEOWNERS (this also affects the standard track exceptions).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

infra Infrastructure issues (npm, GitHub Actions, releases) of this project linter Issues or pull requests regarding the tests / linter of the JSON files. size:l [PR only] 101-1000 LoC changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants