chore: configure rule @typescript-eslint/no-unused-vars

JLarky commented 5 months ago

this allows me to remove lint ignore
vars starting with _ are allowed now (as they should be)

❓ Type of change

[ ] 📖 Documentation (updates to the documentation, readme, or JSdoc annotations)
[ ] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[x] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

📝 Checklist

[ ] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

JLarky commented 5 months ago

pi0 commented 5 months ago

Thanks this is a good idea with ignoring _*! Do you mind to please port it to unjs/eslint-config rules? 🙏🏼

JLarky commented 5 months ago

so I need to split this PR into lint rule changes in that repo and also PR to clean up this stuff, right?

pi0 commented 5 months ago

@JLarky Yes. Actually we have some kind of automation with renovate bot that automatically updates deps (eslint preset) + autofix ci it should automatically removed extra rule. Let's see what happens 🤖

unjs / jiti