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

Added CI and Code Coverage toolings #15

Merged
merged 1 commit into from
May 14, 2021
Merged

Added CI and Code Coverage toolings #15

merged 1 commit into from
May 14, 2021

Conversation

mojoX911
Copy link
Contributor

This adds CI (build + test) using github action.

The tests in nightly build produces coverage report which can be displayed using codecov

Action for maintainer:

Rest is taken care of by the scripts.

This will create CI testing report as well as code coverage report for each PR.

Cargo.lock Outdated Show resolved Hide resolved
@chris-belcher
Copy link
Contributor

Thanks for this. I'll set it up soon.

@mojoX911
Copy link
Contributor Author

mojoX911 commented Apr 24, 2021

Refactored the CI workflow.

This now contains three tasks

  • Continuous Integration (CI)
  • Code Coverage
  • Lint Checking

CI runs are configurable to create more combinations of platform, toolchain and features. Currently it just runs on ubuntu-latest with stable and nightly toolchain using an empty default feature. I have kept the action steps generic so extending different combination is just about adding them to the matrix.

This is now ready for review.

@chris-belcher
Copy link
Contributor

Sorry for not getting back to this sooner. I don't know enough about CI to review this. If you're confident it's fine would you like to set it up? I can add you as another maintainer.

@mojoX911
Copy link
Contributor Author

mojoX911 commented May 8, 2021

No Issues. I wont mind setting it up myself. I think it will help a lot in future merging process.

@chris-belcher chris-belcher merged commit 90e71ba into bitcoin-teleport:master May 14, 2021
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