Skip to content
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

Prerequisites for publishing solidity-coverage 0.8.0 #701

Closed
4 of 5 tasks
cgewecke opened this issue Mar 14, 2022 · 10 comments
Closed
4 of 5 tasks

Prerequisites for publishing solidity-coverage 0.8.0 #701

cgewecke opened this issue Mar 14, 2022 · 10 comments

Comments

@cgewecke
Copy link
Member

cgewecke commented Mar 14, 2022

(This list actively edited)

TODO

@barakman
Copy link

Hi. Is there an ETA on this?

@mejango
Copy link

mejango commented Mar 17, 2022

also curious. thanks, this project has been a massive sanity check as we get ready to deploy.

@cgewecke
Copy link
Member Author

I should have some time for this later this week...

@Nipol
Copy link

Nipol commented Mar 26, 2022

What do you think about viaIR support? In my case, the test case will pass, but the coverage will be 0%.

@mejango
Copy link

mejango commented Mar 28, 2022

updates @cgewecke?
If there's a good chance its not gonna happen, all good, just might as well se the expectation. <3

@cgewecke
Copy link
Member Author

It's going to happen, I'm just running behind sorry.

@cgewecke
Copy link
Member Author

@mejango The beta version is patched and #700 should be resolved for the moment.

@parv3213
Copy link

Hey team! When can we expect support for viaIR?
Meanwhile, is there any workaround to support this locally?

@cgewecke
Copy link
Member Author

@parv3213 There are some tricks to configuring the optimizer if you're hitting stack too deep without viaIR at the link below. (viaIR has to be turned off because it strips out solidity-coverage's instrumentation - see #715)

https://github.com/sc-forks/solidity-coverage/blob/master/docs/faq.md#running-out-of-stack

@barakman
Copy link

barakman commented Sep 7, 2022

Working, thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants