-
-
Notifications
You must be signed in to change notification settings - Fork 158
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 fixes #1363
Lint fixes #1363
Conversation
Signed-off-by: Prabhu Subramanian <[email protected]>
Signed-off-by: Prabhu Subramanian <[email protected]>
@@ -55,6 +55,7 @@ import { | |||
executeParallelGradleProperties, | |||
extractJarArchive, | |||
frameworksList, | |||
generatePixiLockFile, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should have repo tests for pixi with type python which would have caught this
@@ -9882,7 +9882,6 @@ export async function getJarClasses(jarFile) { | |||
"is not recognized as an internal or external command", | |||
) | |||
) { | |||
jarCommandAvailable = false; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Another bug that went unreported
Signed-off-by: Prabhu Subramanian <[email protected]>
Denotests are consistently failing on Windows at the rm -rf command. This needs to be investigated, since the most likely cause must be atom hanging on Windows at a prior test thus blocking the rm command. |
Added more rules such as
noUndeclaredVariables
and found few bugs. Please consider using this biome.json in the new refactor branch and feel free to add more useful rules.