Skip to content

Removing webhint - #5016

Merged
Vidal Ortega (vidorteg) merged 1 commit into
mainfrom
user/vidorteg/remove-webhint
Sep 14, 2026
Merged

Vidal Ortega (vidorteg) merged 1 commit into
mainfrom
user/vidorteg/remove-webhint

Conversation

@vidorteg

Copy link
Copy Markdown
Contributor

Removing webhint as it has also been deprecrated in Edge Devtools 152

Copilot AI lite review requested due to automatic review settings September 14, 2026 18:59

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

No unresolved blocking issues were identified.

Pull request overview

Removes the deprecated webhint integration and related dependencies, configuration, tests, and packaging references.

Changes:

  • Removes webhint lifecycle, diagnostics, telemetry, and utilities.
  • Removes webhint dependencies and configuration.
  • Cleans up tests and packaging entries.
File summaries
File Description
test/helpers/helpers.ts Removes the unused language-client test helper.
test/extension.test.ts Removes webhint mocks.
src/utils.ts Removes webhint utilities.
src/extension.ts Removes webhint integration and lifecycle code.
package.json Removes webhint settings and dependencies.
package-lock.json Removes related locked packages.
.vscodeignore Removes the webhint packaging exception.
Review details
  • Files reviewed: 6/7 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@vidorteg
Vidal Ortega (vidorteg) merged commit f6bb239 into main Sep 14, 2026
8 checks passed
@vidorteg
Vidal Ortega (vidorteg) deleted the user/vidorteg/remove-webhint branch September 14, 2026 22:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants