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

feat(ci): add binary compatiblity test #90

Merged
merged 16 commits into from
Jun 16, 2022
Merged

feat(ci): add binary compatiblity test #90

merged 16 commits into from
Jun 16, 2022

Conversation

tsunrise
Copy link
Contributor

@tsunrise tsunrise commented Jun 8, 2022

Summary

Add test to support binary compatibility of UTXO.

Details

This PR ports the hardcoded test from https://github.com/Manta-Network/Manta/blob/manta/pallets/manta-pay/src/bin/precompute_coins.rs. This test makes sure changes in current PR are still compatible with the UTXO versions in manta-sdk.

Closes: #89


Before we can merge this PR, please make sure that all the following items have been checked off:

  • Linked to an issue with discussion and accepted design OR have an explanation in the PR that describes this work.
  • Added one line describing your change in CHANGELOG.md and added the appropriate changelog label to the PR.
  • Re-reviewed Files changed in the GitHub PR explorer.
  • Checked that changes and commits conform to the standards outlined in CONTRIBUTING.md.

@tsunrise tsunrise changed the title Add binary compatiblity test ci: Add Binary Compatiblity Test Jun 8, 2022
@bhgomes bhgomes added this to the v1.0.0 milestone Jun 9, 2022
@bhgomes bhgomes changed the title ci: Add Binary Compatiblity Test feat(ci): add binary compatiblity test Jun 9, 2022
@tsunrise tsunrise self-assigned this Jun 9, 2022
@tsunrise tsunrise added changelog:added Changelog: add these changes to the `added` section of the changelog norelease labels Jun 9, 2022
tsunrise and others added 4 commits June 9, 2022 13:34
* feat: add documentation for contributing
* fix: update logo spacing and correct formatting for bullets
* feat: update CHANGELOG to include current changes
* fix: use full paths for CHANGELOG and CONTRIBUTING links
* fix: correct Github -> GitHub typo
* feat: add automatic tag-and-release pipeline
* feat: add label descriptions in CONTRIBUTING.md
* fix: update PULL_REQUEST_TEMPLATE links

Signed-off-by: Brandon H. Gomes <bhgomes@pm.me>
stechu
stechu previously requested changes Jun 9, 2022
manta-pay/Cargo.toml Outdated Show resolved Hide resolved
Signed-off-by: Brandon H. Gomes <bhgomes@pm.me>
@bhgomes bhgomes modified the milestones: v1.0.0, v0.6.0 Jun 14, 2022
manta-pay/Cargo.toml Outdated Show resolved Hide resolved
manta-pay/src/test/compat.rs Outdated Show resolved Hide resolved
manta-pay/src/test/compat.rs Outdated Show resolved Hide resolved
manta-pay/src/test/compat.rs Outdated Show resolved Hide resolved
manta-pay/src/test/compat.rs Outdated Show resolved Hide resolved
manta-pay/src/test/compat.rs Outdated Show resolved Hide resolved
manta-pay/src/test/compat.rs Outdated Show resolved Hide resolved
manta-pay/src/test/compat.rs Outdated Show resolved Hide resolved
manta-pay/src/test/compat.rs Outdated Show resolved Hide resolved
manta-pay/src/test/mod.rs Outdated Show resolved Hide resolved
manta-pay/src/test/mod.rs Outdated Show resolved Hide resolved
manta-pay/src/lib.rs Outdated Show resolved Hide resolved
@BoyuanFeng BoyuanFeng self-assigned this Jun 15, 2022
manta-pay/src/lib.rs Outdated Show resolved Hide resolved
manta-pay/src/lib.rs Outdated Show resolved Hide resolved
manta-pay/Cargo.toml Outdated Show resolved Hide resolved
@bhgomes bhgomes dismissed stechu’s stale review June 16, 2022 20:44

we resolved the issue

@BoyuanFeng BoyuanFeng merged commit dd1b3eb into main Jun 16, 2022
@BoyuanFeng BoyuanFeng deleted the ci/bin-compat branch June 16, 2022 20:48
@bhgomes bhgomes modified the milestones: v0.6.0, v0.5.1 Jun 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog:added Changelog: add these changes to the `added` section of the changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Binary Compatability Test
4 participants