Fix/security - Githubissues

sushiswap / shoyu

Shoyu contracts

https://shoyunft.com

MIT License

168 stars 122 forks source link

Fix/security #32

Closed TheGreatHB closed 3 years ago

coveralls commented 3 years ago

Pull Request Test Coverage Report for Build 1163438562

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 10 (0.0%) changed or added relevant lines in 1 file are covered.
33 unchanged lines in 2 files lost coverage.
Overall coverage remained the same at 0.0%

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
contracts/ERC721GovernanceToken.sol	0	10	0.0%
<!--	Total:	0	10	0.0%	-->

Files with Coverage Reduction	New Missed Lines	%
contracts/ERC721GovernanceToken.sol	2	0.0%
contracts/base/BaseExchange.sol	31	0.0%
<!--	Total:	33		-->

Totals
Change from base Build 1162574690:	0.0%
Covered Lines:	0
Relevant Lines:	811

💛 - Coveralls

levx-me commented 3 years ago

We need to reconsider whether to keep ERC721GovernanceToken and OrderBook