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

Does not work with new features in solidity 0.8.18 (named mapping parameters) #782

Closed
cyberhorsey opened this issue Feb 15, 2023 · 4 comments

Comments

@cyberhorsey
Copy link

cyberhorsey commented Feb 15, 2023

Running coverage with solc ^0.8.18 gives:

 Could not instrument: input 'addr' expecting '=>'

This may be an issue with the solidity-parser plugin though?

@cyberhorsey cyberhorsey changed the title Does nto work with new features in solidity 0.8.18 (named mapping parameters) Does not work with new features in solidity 0.8.18 (named mapping parameters) Feb 15, 2023
@cyberhorsey
Copy link
Author

I opened a PR for this.
#783

@eladiosch
Copy link

Is there an ETA for this? The fix looks good, so please merge so we can continue to use this tool with the new version of Solidity

@HardlyDifficult
Copy link

Workaround: if you are using yarn, in your package.json add this and then run yarn again.

  "resolutions": {
    "@solidity-parser/parser": "0.16.0"
  },

I believe npm can do the same with overrides but have not tested that.

@cgewecke
Copy link
Member

Resolved in 0.8.4

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

4 participants