[go: nahoru, domu]

Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(typings): Restore inadvertently-deleted typings/msg/msg.d.ts; add TS import test #7955

Merged
merged 2 commits into from
Mar 21, 2024

Conversation

cpcallen
Copy link
Contributor

The details

Resolves

Fixes #7952

Proposed Changes

Restore main typings file for langfiles, typings/msg/msg.d.ts, which was inadvertently deleted in PR #7822.

Reason for Changes

Imports of blockly/msg/* are broken; this fixes them.

Test Coverage

Added a test that would have caught this mistake had it existed before now.

Additional Information

The deletion was part of a well intentioned attempt to remove spurious files from typings/msg/ that did not correspond to published langfiles, but msg.d.ts should have been retained because it is reexported by all the other *.d.ts files in this directory.

Fixes google#7952.

The main typings file for langfiles, typings/msg/msg.d.ts, was
inadvertently deleted in PR google#7822.

This was part of a well intentioned attempt to remove spurious
files from typings/msg/ that do not correspond to published
langfiles, but this file should have been retained because
msg.d.ts is reexported by all the other *.d.ts files in this
directory.
@cpcallen cpcallen requested a review from a team as a code owner March 21, 2024 12:44
@github-actions github-actions bot added the PR: fix Fixes a bug label Mar 21, 2024
@cpcallen cpcallen merged commit d3575ad into google:rc/v11.0.0 Mar 21, 2024
10 checks passed
@cpcallen cpcallen deleted the fix/7952 branch March 21, 2024 15:01
@cpcallen cpcallen linked an issue Mar 21, 2024 that may be closed by this pull request
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: fix Fixes a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

11.0.0-beta.4 can't import languages in TypeScript
2 participants