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

Revert reason in the result of contract's method call #3536

Merged
merged 1 commit into from
Dec 24, 2020

Conversation

vbaranov
Copy link
Member

@vbaranov vbaranov commented Dec 24, 2020

Motivation

Display revert reason in the result of the contract method call.

Changelog

Screenshot 2020-12-24 at 13 35 02

Checklist for your Pull Request (PR)

@vbaranov vbaranov changed the title Revert reason in result of contract call Revert reason in the result of contract's method call Dec 24, 2020
@coveralls
Copy link

coveralls commented Dec 24, 2020

Pull Request Test Coverage Report for Build 06c87bfcfeca533a29ec41b972ca99d131ae3251-PR-3536

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 7 unchanged lines in 3 files lost coverage.
  • Overall coverage decreased (-0.9%) to 66.838%

Files with Coverage Reduction New Missed Lines %
lib/block_scout_web/controllers/chain/market_history_chart_controller.ex 1 78.57%
test/block_scout_web/features/pages/chain_page.ex 1 90.0%
lib/block_scout_web/controllers/smart_contract_controller.ex 5 80.65%
Totals Coverage Status
Change from base Build 1c926d630d5bc2e0a551276e61ab9fd0b2872188: -0.9%
Covered Lines: 2078
Relevant Lines: 3109

💛 - Coveralls

@vbaranov vbaranov merged commit 46a5d28 into master Dec 24, 2020
@vbaranov vbaranov deleted the vb-revert-reason-in-call branch December 24, 2020 12:33
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

Successfully merging this pull request may close these issues.

None yet

2 participants