rust-bitcoin / rust-bitcoin / 13189986790

83%

push

github

Merge rust-bitcoin/rust-bitcoin#4013: CI: Remove the API job

<a class=hub.com/rust-bitcoin/rust-bitcoin/commit/2eccb3897a90221c4302947adc10a1e370e0d4a5">2eccb3897 CI: Remove the API job (Tobin C. Harding)

Pull request description:

  It has been argued that the API files provide value for verifying that code changes produce API changes in the expected way. This helps devs and reviewers.

  It has also been argued that the CI job adds friction and forces devs to configure their environment unnecessarily.

  Both viewpoints have some merit.

  Lets remove the API files for now, and when we start doing 1.0-alpha releases and can start to expect minimal API breaking changes (and therefore minimal need to use the tooling)

ACKs for top commit:
  Kixunil:
    ACK 2eccb3897a90221c4302947adc10a1e370e0d4a5

Tree-SHA512: 566edf6cd0a1fd481b2444a4f1784de4b8c

20658 of 25042 relevant lines covered (82.49%)

17373.34 hits per line