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

Add Github CI check #96

Open
wants to merge 45 commits into
base: straka/arkworks_minus_cli
Choose a base branch
from
Open

Add Github CI check #96

wants to merge 45 commits into from

Conversation

hanyunx
Copy link
Contributor

@hanyunx hanyunx commented Apr 30, 2022

Add github workflow for CI check replacing CircleCI. There are three parts in the workflow:

  1. CI Test: Run all tests in crates/ and generate test coverage report
  2. Check and Lint: Run cargo check/fmt/clippy for lint check
  3. Contract and Cross Platform:
    a. run all tests for smart contracts in solidity
    b. build crates/threshold-bls-ffi/cross on both macOS and Android environment ❌
    c. build wasm in crates/threshold-bls-ffi

@codecov-commenter
Copy link

codecov-commenter commented May 6, 2022

Codecov Report

❗ No coverage uploaded for pull request base (straka/arkworks_minus_cli@10a9ec8). Click here to learn what that means.
The diff coverage is n/a.

@@                     Coverage Diff                      @@
##             straka/arkworks_minus_cli      #96   +/-   ##
============================================================
  Coverage                             ?   65.11%           
============================================================
  Files                                ?       20           
  Lines                                ?     1608           
  Branches                             ?        0           
============================================================
  Hits                                 ?     1047           
  Misses                               ?      561           
  Partials                             ?        0           

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 10a9ec8...4a2c24a. Read the comment docs.

@hanyunx hanyunx changed the title Add CI check Add Github CI check May 6, 2022
@hanyunx
Copy link
Contributor Author

hanyunx commented May 6, 2022

In .github/workflow/contracts-and-cross.yaml, the scripts of testing smart contracts and building iOS & Android are commented for now because of failing to build.

@hanyunx hanyunx requested a review from mstraka100 May 6, 2022 07:05
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.

2 participants