76331ae Add a just cmd to check for API changes (Tobin C. Harding) b222f40 CI: Add job to check for API changes (Tobin C. Harding) 9e7cd97 Add a script to check the public API (Tobin C. Harding) Pull request description: This PR is rust-bitcoin#1880 re-opened. Add a script that checks the public API of `hashes` and `bitcoin`. Document how to use it during development. Call it in CI. Do not add it to githooks because the githooks because its expected to be run per PR not per commit. Includes a `just` command to run the script: `just check-api` ### Implied workflow change This PR imposes workflow changes. Explicitly: all PRs that change the public API of `bitcoin`, `base58`, `hashes`, `io`, or `units` must contain changes to the api text files. Suggestion: We add the patch updating api text files as a separate patch at the end of each PR so we can haggle over it separately from the actual code changes. Fix: rust-bitcoin#1875 ACKs for top commit: apoelstra: ACK 76331ae normally would complain about the whitespace but I would like to ACK/merge this quickly since most other PR that gets merged will force it to be rebased. also will one-ACK merge as "no NACKs or comments from maintainers in 2 weeks" Tree-SHA512: 67b20e2ce0c22aa67be931c4da0b591bc351ccb1aa620003c60bfb4b10d5c292edceca929bf6318993f2d16f9f58374aac336adf0f8234c5e2f16e3857b7901b

Check for API breaking changes in CI #1875

Description

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions