Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

Add fallback check in ABI validation #3459

Merged
merged 1 commit into from
Nov 16, 2016
Merged

Add fallback check in ABI validation #3459

merged 1 commit into from
Nov 16, 2016

Conversation

ngotchac
Copy link
Contributor

Fixes #3392

Test with

contract test {
  function() {}
}

For now, the ABI validation ignores the fallback function. Not sure how it can be used.

@ngotchac ngotchac added A0-pleasereview 🤓 Pull request needs code review. M7-ui labels Nov 15, 2016
@jacogr jacogr added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Nov 15, 2016
@coveralls
Copy link

Coverage Status

Coverage remained the same at 86.012% when pulling 3b8286f on ng-abi-fallback into 039bd3c on master.

@gavofyork gavofyork merged commit 4d5ce8f into master Nov 16, 2016
@gavofyork gavofyork deleted the ng-abi-fallback branch November 16, 2016 03:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A8-looksgood 🦄 Pull request is reviewed well.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants