-
Notifications
You must be signed in to change notification settings - Fork 47
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
[Feature]: optimize the ecma version check. #532
Comments
|
@nyqykk How about align options with rsbuild-plugin-check-syntax: https://github.com/rspack-contrib/rsbuild-plugin-check-syntax?tab=readme-ov-file#options |
Good suggestion
|
we will add this feature after refactor Overview UI #615 |
I's better to directly use |
yes, our current plan is like this.
|
support in v1.0 #655 |
Can it support a specific syntax level? For example, Chrome 70? Through syntax checking, of course, this may be a hard job, and I haven’t seen any actual cases to do this. This is more intuitive and more useful |
ofcource it supports that,like this https://github.com/rspack-contrib/rsbuild-plugin-check-syntax?tab=readme-ov-file#options |
Great feature! Can you list all the problems? For example, if I target chrome 70, can you list all the syntax checks that exceed chrome 70 to form a list. This will make it easier for me to modify it. Instead of just handling one and throwing an exception to terminate the whole process In fact, most of the syntax outside the target is caused by skipping swc transform, such as npm package code under node_modules. I hope to be able to use this method to check and set the appropriate target and include. This is what rsdoctor can provide |
It will list all files exceed syntax check but just throw the first error in each file even if it has multiple errors. Only by solving the first problem can the second problem be scanned |
What problem does this feature solve?
What does the proposed API look like?
The text was updated successfully, but these errors were encountered: