MTP verification wrap up - Githubissues

ChainSafe / chainbridge-celo

GNU Lesser General Public License v3.0

3 stars 2 forks source link

MTP verification wrap up #129

Open P1sar opened 3 years ago

P1sar commented 3 years ago

This PR merges DEVELOP branch into MAIN.

Description

Related Issue Or Context

Closes: #128

How Has This Been Tested? Testing details.

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation

Checklist:

[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] I have ensured that all acceptance criteria (or expected behavior) from issue are met
[ ] I have updated the documentation locally and in chainbridge-docs.
[ ] I have added tests to cover my changes.
[ ] I have ensured that all the checks are passing and green, I've signed the CLA bot

lgtm-com[bot] commented 3 years ago

This pull request introduces 2 alerts when merging c4954b0b51def5d0fd968c5177b7c823e7ab7cb3 into 3b0dc76b9e660bc5d2a41d41baa0036822295cab - view on LGTM.com

new alerts:

2 for Useless assignment to local variable