Skip to content

fix: normalize all JSDoc comments#335

Open
MattiasBuelens wants to merge 1 commit into
open-wc:masterfrom
MattiasBuelens:normalize-jsdoc-comments
Open

fix: normalize all JSDoc comments#335
MattiasBuelens wants to merge 1 commit into
open-wc:masterfrom
MattiasBuelens:normalize-jsdoc-comments

Conversation

@MattiasBuelens

Copy link
Copy Markdown

Any JSDocTag's comment can hold either a plain string or a NodeArray<JSDocComment>. We need to always normalize these to a plain string, otherwise the generated manifest would still contain references to TypeScript nodes and source files which cannot be serialized to JSON.

@netlify

netlify Bot commented Apr 24, 2026

Copy link
Copy Markdown

Deploy Preview for custom-elements-manifest-analyzer ready!

Name Link
🔨 Latest commit 976d3d6
🔍 Latest deploy log https://app.netlify.com/projects/custom-elements-manifest-analyzer/deploys/69eb5ccb48bb9500089a6b4a
😎 Deploy Preview https://deploy-preview-335--custom-elements-manifest-analyzer.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@MattiasBuelens MattiasBuelens changed the title Normalize all JSDoc comments fix: normalize all JSDoc comments Apr 24, 2026
MattiasBuelens added a commit to THEOplayer/web-ui that referenced this pull request Apr 28, 2026
MattiasBuelens added a commit to THEOplayer/web-ui that referenced this pull request Apr 28, 2026
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.

1 participant