Skip to content
New issue

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

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

Already on GitHub? # to your account

fix(check): properly surface dependency errors in types file of js file #25860

Merged

Conversation

dsherret
Copy link
Member

@dsherret dsherret commented Sep 25, 2024

We weren't surfacing dependency errors in types files of js files.

"args": "run --unstable-sloppy-imports --check main.ts",
"output": "sloppy.out"
}
}
Copy link
Member Author

Choose a reason for hiding this comment

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

This was during my investigation of the issue. I decided to keep it because it's a better test.

Copy link
Member

@bartlomieju bartlomieju left a comment

Choose a reason for hiding this comment

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

LGTM

@dsherret dsherret merged commit 8cdb309 into denoland:main Sep 25, 2024
17 checks passed
@dsherret dsherret deleted the fix_properly_surface_error_dts_for_js_file branch September 25, 2024 13:05
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants