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

ci: add workspace or single package to build-test #1575

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

oleonardolima
Copy link
Contributor

fixes #1018

Description

I was going through the Project board and gave it a try as I'm already working with CI on rust-esplora-client, and it looked simple.

I was wondering if it should really be another job, I felt that adding to the matrix strategy it's simpler and clearer, let me know what you think.

Notes to the reviewers

I'm happy with either a new job or this approach (if it's sound), please let me know of any flaws with this one.

Changelog notice

  • Adds --workspace and -p <bdk-package> to CI build-test job matrix strategy, to build and test the workspace and every single BDK package.

Checklists

All Submissions:

  • I've signed all my commits
  • I followed the contribution guidelines
  • I ran cargo fmt and cargo clippy before committing

It adds a new choice for the matrix strategy, being `--workspace` or a
single package, i.e. `-p bdk_wallet`. It will then build and test for
the whole workspace and each single package.
@oleonardolima
Copy link
Contributor Author

Oh, wow! It really increase the number of checks on CI, I might rethink the approach.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

Add CI tests for building individual packages
1 participant