Closed Quazia closed 1 year ago
Can you also add solidity
to .vscode/extensions.json
and a .solhint
file to the repo and "solidity.linter": "solhint",
to .vscode/settings.json
for us vscode users.
Also, while we are at let's move over to forge for formatting and set up the vscode workspace for it too: https://book.getfoundry.sh/config/vscode#3-formatter
Definitely on both counts
Closed by #173
Right now it's up to the developer to enforce linting on the code they submit. For this issue we want to ensure that linting is checked on each PR using
solhint
before it can be merged into main using GitHub actions.