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

[lint] Add eslint-plugin-import to detect AMD or commonjs imports #7339

Open
unlikelyzero opened this issue Jan 4, 2024 · 0 comments
Open
Labels
type:maintenance tests, chores, or project maintenance

Comments

@unlikelyzero
Copy link
Contributor

Summary

#7331 has merged and we should protect against AMD and commonjs syntax.

https://www.npmjs.com/package/eslint-plugin-import should work

@unlikelyzero unlikelyzero added the type:maintenance tests, chores, or project maintenance label Jan 4, 2024
@ozyx ozyx self-assigned this Jan 4, 2024
@ozyx ozyx added this to the Target:3.3.0 milestone Jan 4, 2024
@unlikelyzero unlikelyzero modified the milestones: Target:3.3.0, Target:4.0.0 Jan 17, 2024
@akhenry akhenry removed this from the Target:4.0.0 milestone Mar 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:maintenance tests, chores, or project maintenance
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants