Skip to content

Conversation

@Bashamega
Copy link
Contributor

I have updated it because it was misleading

@github-actions
Copy link
Contributor

Thanks for the PR!

This section of the codebase is owned by @saschanaz - if they write a comment saying "LGTM" then it will be merged.

- `removedTypes.jsonc`: types that are defined in the spec but should be removed.
- `comments.json`: comment strings to be embedded in the generated .js files.
- `deprecatedMessage.json`: the reason why one type is deprecated. The reason why it is a separate file rather than merge in comment.json is mdn/apiDescriptions.json would also possibly be deprecated.
- `comments.json`: comment strings to be embedded in the generated .d.ts files.
Copy link
Collaborator

Choose a reason for hiding this comment

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

(I think this is mostly deprecated after your MDN patch. We should consider removing it)

@saschanaz
Copy link
Collaborator

LGTM

@github-actions
Copy link
Contributor

There was an issue merging, maybe try again saschanaz. Details

@jakebailey jakebailey merged commit 7ca5f39 into microsoft:main Apr 24, 2025
7 checks passed
@Bashamega Bashamega deleted the fix-readme branch November 3, 2025 04:18
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