Skip to content

Commit c12a81b

Browse files
committed
Merge #64: Introduce exposed API check
6b70a27 Add the exposed API check to the existing Lint CI (Nick Johnson) ddf7c06 Initialize the api files (Nick Johnson) Pull request description: I've appreciated this check over in `rust-bitcoin` and think it can help here as well. Initialing the api files in the first commit by running `cargo rbmt api`. Added the check to CI by extending the existing Lint job, I don't think we need to go testing the 20 parallel jobs limit anytime soon since these are all pretty quick. Bet let me know if there is a preferred job breakdown. ACKs for top commit: tcharding: ACK 6b70a27 Tree-SHA512: 276deb25122050d94049d836eea0b4f29a09da07afa92db29a45ddeaeef1bc9927e8a8c5afe1675c75b3b973c49e9f6e8a715e29b125846bbd440416e568dafa
2 parents 2f26bb2 + 6b70a27 commit c12a81b

File tree

4 files changed

+9157
-0
lines changed

4 files changed

+9157
-0
lines changed

.github/workflows/rust.yml

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -68,6 +68,18 @@ jobs:
6868
- name: "Run lint"
6969
run: cargo +${{ steps.setup.outputs.nightly-version }} rbmt lint
7070

71+
API:
72+
name: API check
73+
runs-on: ubuntu-latest
74+
strategy:
75+
fail-fast: false
76+
steps:
77+
- uses: actions/checkout@v6
78+
- uses: rust-bitcoin/rust-bitcoin-maintainer-tools/.github/actions/setup-rbmt@master
79+
id: setup
80+
- name: "Run API check"
81+
run: cargo +${{ steps.setup.outputs.nightly-version }} rbmt api
82+
7183
Docs:
7284
name: Docs - stable toolchain
7385
runs-on: ubuntu-latest

0 commit comments

Comments
 (0)