feat(hardhat-zksync-solc): support suppressed errors and warnings.

[mmv08]

What 💻

• Support suppressedErrors and suppressedWarningscompiler options in hardhat-zksync-solcconfig

Why ✋

• To compile older contracts that may've been written without EraVM in mind.

Evidence 📷

I added a test that verifies that the changes work as intended. It's currently failing because of the pending compiler fix.

Notes 📝

• This PR is a draft of the proposed changes. The final PR depends on fix: correctly deserialize suppressed_warnings and suppressed-errors from standard json input era-compiler-solidity#146, and the final (plus any proposed changes during the PR review) changes will be applied after the new compiler release.

[mmv08]

For some reason, I encountered some package manager conflict during the pre-commit hook and couldn't create a commit. It was invoking yarn, but then it'd fail with an error that the package manager used in the project is pnpm. I couldn't find yarn used anywhere in the project, so I guess pnpm is the one to use here

[kiriyaga-txfusion]

Hey @mmv08, we’ve implemented this feature in this PR, and it’s ready for merging and release. Please check it out.

[mmv08]

hey @kiriyaga-txfusion, the PR looks good, but I'm not sure it'll work with this change: matter-labs/era-compiler-solidity#146. Perhaps it needs an update.

I like how the other PR was done under the hood, but I'm not a big fan of the user-facing part. If something can be specified as a string, I'm unsure why one must import an enum.

[kiriyaga-txfusion]

hey @kiriyaga-txfusion, the PR looks good, but I'm not sure it'll work with this change: matter-labs/era-compiler-solidity#146. Perhaps it needs an update.

I like how the other PR was done under the hood, but I'm not a big fan of the user-facing part. If something can be specified as a string, I'm unsure why one must import an enum.

Thanks for pointing that out. The PR needs to be updated to handle changes in zksolc. I'll update the parameters to be strings so users don’t have to import anything. We’ll be releasing it this week, so stay tuned! :D