Josselin Feist

Results 134 comments of Josselin Feist

This PR has been on hold because the breaking change might impact other projects. Slither has an open PR https://github.com/crytic/slither/pull/889, but the related code has changed so it needs to...

I am not sure about this solution. It looks like we are using a short term solution for a non-critical/blocking fix, that we know will require a refactoring. If we...

Hey @mds1. It might be due to https://github.com/foundry-rs/foundry/issues/1646. A quick workaround is to add a fake contract (`contract Fake{}`) in the file listed in the `ValueError`. Let me know if...

We can add a section on known fixes, like - Remove the compilation directory and try again - If it seems stuck, try to compile it outside of our tools,...

Related to https://github.com/crytic/crytic-compile/issues/79. Adding a `--additional-directory` flag could fix the issue

Hi @yxliang01, Do you have more context regarding "locally installed" libraries accessible by solc? Is it for the ones used through the remapping, or is it something else?

What version of npx/node are you using? Did you change any of the default configuration? I don't have this confirmation (`Ok to proceed? (y)`) needed in my local tests

Hi @endorphin, Good catch and sorry for this mistake. We will release a new version in the upcoming days. Note that the version on pip is `e45c759`, only the tag/release...

Hi @yxliang01 , Do you have an example that you could share? If I understand correctly, you want to give to `solc_args` either a string, or a list of strings,...

`saddle` does not provide the AST info without manual configuration. I opended an issue in saddle (https://github.com/compound-finance/saddle/issues/35) to see if we can add the info to the default compilation output.