Skip to content

[#62] Update octocrab to 0.19 - #63

Merged
heitor-lassarote merged 1 commit into
masterfrom
heitor-lassarote/#62-octocrab-0.19
Oct 14, 2026
Merged

heitor-lassarote merged 1 commit into
masterfrom
heitor-lassarote/#62-octocrab-0.19

Conversation

@heitor-lassarote

@heitor-lassarote heitor-lassarote commented Sep 25, 2026 •

Copy link
Copy Markdown
Member

Description

Updates octocrab from 0.18.1 to 0.19. 0.18.1 depends on hyperx, which caps bytes below 1.2, and without the committed Cargo.lock that makes combine fail to build. cargo install update-daemon without --locked would hit this once the crate is published. 0.19 no longer depends on hyperx, and no code changes are needed.

This is stacked on #61 (and so on #59). Once those are merged, this PR's base becomes master.

Related issue(s)

Resolves #62

Notes for reviewers

  • With Cargo.lock removed, a fresh resolution picks bytes 1.12.1, contains no hyperx, and builds. With the resolver change from [#56] Move to upstream ssh2-config 0.8 #59 it also stays on versions compatible with Rust 1.88.
  • nix build, which runs the tests and pedantic clippy, passes.
  • One behaviour change: 0.19 retries requests that get a 401, up to 3 attempts in total, also with a personal token. 0.18.1 did not retry personal-token requests. With an expired GITHUB_TOKEN, each call is now sent 3 times before it fails with the same error.

@heitor-lassarote
heitor-lassarote marked this pull request as ready for review September 25, 2026 22:29
@heitor-lassarote heitor-lassarote self-assigned this Sep 25, 2026
@heitor-lassarote
heitor-lassarote force-pushed the heitor-lassarote/#62-octocrab-0.19 branch from 9fdab12 to 68da6e2 Compare September 28, 2026 14:47
@heitor-lassarote
heitor-lassarote force-pushed the heitor-lassarote/#62-octocrab-0.19 branch from 68da6e2 to 75a424f Compare September 28, 2026 18:06
@heitor-lassarote
heitor-lassarote force-pushed the heitor-lassarote/#62-octocrab-0.19 branch 2 times, most recently from ead65c2 to a4ac4da Compare October 14, 2026 13:39
@heitor-lassarote
heitor-lassarote force-pushed the heitor-lassarote/#62-octocrab-0.19 branch from a4ac4da to 8dfdef2 Compare October 14, 2026 13:48
Base automatically changed from heitor-lassarote/#60-gpgme-0.11 to master October 14, 2026 13:49
Problem: `octocrab` 0.18.1 depends on `hyperx` 1.4.0, which requires
`bytes` below 1.2. Without the committed Cargo.lock, `combine` 4.6.8
is picked through `gitlab`, and it needs a newer `bytes`, so the build
fails. `cargo install update-daemon` without `--locked` resolves
dependencies this way.

Solution: update `octocrab` to 0.19, which no longer depends on
`hyperx`. No code changes are needed. In the lockfile, `hyperx`,
`base64` 0.20, `language-tags` and `unicase` go away, and
`tracing-attributes` is added.
@heitor-lassarote
heitor-lassarote force-pushed the heitor-lassarote/#62-octocrab-0.19 branch from 8dfdef2 to f7884ed Compare October 14, 2026 13:49
@heitor-lassarote
heitor-lassarote merged commit 1926913 into master Oct 14, 2026
1 check passed
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.

Building without Cargo.lock fails: octocrab 0.18 caps bytes below what combine needs

2 participants