Skip to content

Remove non-existing sourcemap reference dompurify and marked#278154

Merged
mjbvz merged 1 commit intomainfrom
hediet/b/passive-beaver
Nov 18, 2025
Merged

Remove non-existing sourcemap reference dompurify and marked#278154
mjbvz merged 1 commit intomainfrom
hediet/b/passive-beaver

Conversation

@hediet
Copy link
Member

@hediet hediet commented Nov 18, 2025

No description provided.

Copilot AI review requested due to automatic review settings November 18, 2025 17:15
@hediet hediet self-assigned this Nov 18, 2025
@vs-code-engineering
Copy link

📬 CODENOTIFY

The following users are being notified based on files changed in this PR:

@bpasero

Matched files:

  • src/vs/base/browser/dompurify/dompurify.js

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR removes sourcemap URL references from two vendored third-party JavaScript libraries (marked and DOMPurify) that point to non-existent .map files, cleaning up broken references in the codebase.

  • Removed //# sourceMappingURL=marked.esm.js.map comment from marked.js
  • Removed //# sourceMappingURL=purify.es.mjs.map comment from dompurify.js

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
src/vs/base/common/marked/marked.js Removed sourcemap reference to non-existent marked.esm.js.map file
src/vs/base/browser/dompurify/dompurify.js Removed sourcemap reference to non-existent purify.es.mjs.map file

@mjbvz
Copy link
Collaborator

mjbvz commented Nov 18, 2025

Thanks!

@mjbvz mjbvz merged commit 83bca56 into main Nov 18, 2025
33 of 34 checks passed
@mjbvz mjbvz deleted the hediet/b/passive-beaver branch November 18, 2025 18:47
@vs-code-engineering vs-code-engineering bot locked and limited conversation to collaborators Jan 2, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants