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

feat: add new diag "namespace alias" in javascript code #1196

Conversation

UnfairBots
Copy link
Contributor

Summary: Add new diag E0719, using Typescript namespace alias in javascript code to separate E0274, rename some test functions from import_alias to namespace_alias

This closes #1138

Copy link

github-actions bot commented Jan 27, 2024

CLA Assistant Lite bot Thank you for your contribution! Like many free software projects, you must sign our Contributor License Agreement before we can accept your contribution.

EDIT: All contributors have signed quick-lint-js' Contributor License Agreement (CLA-v1.md).

@UnfairBots
Copy link
Contributor Author

I have read and hereby agree to quick-lint-js' Contributor License Agreement (CLA-v1.md).

@UnfairBots
Copy link
Contributor Author

recheck

@UnfairBots UnfairBots force-pushed the rename_import_alias_namespace_alias_add_new_diag_namespace_alias branch from d1d27a6 to 9e773ad Compare January 27, 2024 17:17
Copy link
Collaborator

@strager strager left a comment

Choose a reason for hiding this comment

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

This patch is fantastic! Thanks a lot. Sorry for the delay in reviewing.

Let me know when I should merge it.

src/quick-lint-js/fe/parse-statement.cpp Outdated Show resolved Hide resolved
src/quick-lint-js/fe/parse-statement.cpp Outdated Show resolved Hide resolved
…script code to separate E0274, rename some test functions from import_alias to namespace_alias

This closes quick-lint#1138
@UnfairBots UnfairBots force-pushed the rename_import_alias_namespace_alias_add_new_diag_namespace_alias branch from 963b20d to 89ed8df Compare February 27, 2024 14:56
@UnfairBots UnfairBots requested a review from strager February 27, 2024 15:01
@UnfairBots
Copy link
Contributor Author

Hello @strager, I updated new patch and resolved your comments, please review and let me know if any change is required.
Thanks

@strager
Copy link
Collaborator

strager commented Mar 16, 2024

Landed as Git commit 24040d4. Thanks again!

@strager strager closed this Mar 16, 2024
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.

refactor: rename import_alias -> namespace_alias
2 participants