Skip to content
This repository has been archived by the owner on Jan 24, 2022. It is now read-only.

Update ABI methods listing for solc 0.6 #1362

Merged
merged 4 commits into from
Jan 3, 2020
Merged

Conversation

spalladino
Copy link
Contributor

Solidity 0.6 removes the constant attribute in the ABI specification, in favour of the stateMutability one. This PR uses constant if found, or stateMutability otherwise.

Fixes #1350

Copy link
Contributor

@nventuro nventuro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

packages/lib/src/artifacts/Contract.ts Outdated Show resolved Hide resolved
Co-Authored-By: Nicolás Venturo <[email protected]>
@spalladino spalladino added the status:ready-to-merge Order mergify to merge label Jan 3, 2020
@mergify mergify bot merged commit 08f7149 into master Jan 3, 2020
@mergify mergify bot deleted the fix/sendtx-solc-0.6 branch January 3, 2020 20:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
status:ready-to-merge Order mergify to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Solidity 0.6: oz send-tx Error: A method name must be specified.
2 participants