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: update node package call from #42

Merged
merged 6 commits into from
May 29, 2024
Merged

Conversation

BBboy01
Copy link
Contributor

@BBboy01 BBboy01 commented May 29, 2024

Checklist (adding a new tool):

  • If it's a linter it was exported in linter/init.lua
  • The tool was added to the README
  • I do not wish to write a test, but I can confirm that it works on my machine OR I have written corresponding tests for the tools I added. (don't worry if you didn't write a test, it's just nice to see the green check mark)

In frontend project, linter and formatter with different versions defined in package.json, we need use npx to use project scope script to run instead of the global one.

eslint v9 only support eslint.config.js, eslint.config.cjs and eslint.config.mjs

@BBboy01
Copy link
Contributor Author

BBboy01 commented May 29, 2024

seems like flake_8 diagnostic format differed as version update, I fixed it btw

@BBboy01
Copy link
Contributor Author

BBboy01 commented May 29, 2024

luacheck broken too...

@xiaoshihou514
Copy link
Collaborator

I will fix detekt tests some time later...

@xiaoshihou514 xiaoshihou514 merged commit a9bb8a5 into nvimdev:main May 29, 2024
1 of 2 checks passed
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.

2 participants