-
Notifications
You must be signed in to change notification settings - Fork 6.3k
Add Slippy to the list of Solidity tools #16193
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
base: develop
Are you sure you want to change the base?
Conversation
|
Thank you for your contribution to the Solidity compiler! A team member will follow up shortly. If you haven't read our contributing guidelines and our review checklist before, please do it now, this makes the reviewing process and accepting your contribution smoother. If you have any questions or need our help, feel free to post them in the PR or talk to us directly on the #solidity-dev channel on Matrix. |
mehtavishwa30
left a comment
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.
Thanks for your contributions to Solidity resources page, @fvictorio. We curate all the resources here carefully based on maturity, usage, and interest around the tooling in the Solidity community and ecosystem.
Since Slippy is a relatively new tool, we would like to take some time to observe how it grows and add it later based on feedback from our surveys. I will be closing this PR on the above grounds for now.
|
Hi @mehtavishwa30, it's been a couple of months now. Slippy has been adopted by a few serious projects and popular templates and I'm very actively maintaining it. Would you be open to including it now? Btw, this list also includes ethlint, which hasn't had a new version in six years (!), doesn't work with new versions of Solidity, and basically no one uses. Maybe Slippy doesn't have the level of adoption that it needs to be included, but in that case I think ethlint should be dropped from the list, because it's clearly a worse tool to recommend to users. |
|
Hey @fvictorio, we revisited the topic internally and will merge your suggestion. Although it seems like our spellchecker is tripping up over |
|
Done, thanks a lot for reconsidering! |
| compilability | ||
| keypair | ||
| wast | ||
| Slippy |
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.
| Slippy | |
| slippy |
For some reason chk_spelling job is still failing. It's reporting a lower-case variant there, so maybe this will fix the problem.
Added Slippy, a Solidity linter I'm working on, to the list of Solidity tools.