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

fix: remove unneeded ERC20Upgradeable inheritance [N-17] #620

Merged

Conversation

pcarranzav
Copy link
Member

No description provided.

@pcarranzav
Copy link
Member Author

Broken test seems to be an instance of #576 - so retrying

@codecov
Copy link

codecov bot commented Jul 14, 2022

Codecov Report

Merging #620 (1e171db) into pcv/571-n16-typos (012e3d2) will not change coverage.
The diff coverage is n/a.

@@                Coverage Diff                 @@
##           pcv/571-n16-typos     #620   +/-   ##
==================================================
  Coverage              91.91%   91.91%           
==================================================
  Files                     44       44           
  Lines                   2079     2079           
  Branches                 354      354           
==================================================
  Hits                    1911     1911           
  Misses                   168      168           
Flag Coverage Δ
unittests 91.91% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
contracts/l2/token/GraphTokenUpgradeable.sol 100.00% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 012e3d2...1e171db. Read the comment docs.

Base automatically changed from pcv/571-n16-typos to pcv/arb-full-deployment July 26, 2022 14:53
@pcarranzav pcarranzav merged commit 1e171db into pcv/arb-full-deployment Jul 26, 2022
@pcarranzav pcarranzav deleted the pcv/571-n17-unnecessary-inheritance branch July 26, 2022 14:54
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