Skip to content

Update to rustc (to nightly-2026-05-08) and fix#947

Merged
apoelstra merged 3 commits into
rust-bitcoin:masterfrom
tcharding:push-vwnvytpuuoxk
May 20, 2026
Merged

Update to rustc (to nightly-2026-05-08) and fix#947
apoelstra merged 3 commits into
rust-bitcoin:masterfrom
tcharding:push-vwnvytpuuoxk

Conversation

@tcharding
Copy link
Copy Markdown
Member

@tcharding tcharding commented May 14, 2026

Use the bot patch from #939 then:

  • Do trivial lint fixes
  • Update the cron job because it says 'daily'

Update Nightly Rustc Bot and others added 3 commits May 9, 2026 03:49
Found by the linter when upgrading to recent nightly toolchain.
We only update the nightly toolchain weekly but in a couple of places
we say 'daily'.
Copy link
Copy Markdown
Contributor

@trevarj trevarj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Heh. ACK

@tcharding
Copy link
Copy Markdown
Member Author

If you'd like your acks to go into the merge commit @trevarj you can use ACK <commit hash of tip>. Andrew's merge scripts will then pick up that post and add it to the merge commit. Then you can be git blamed for letting bugs past review and fun stuff like that ...

@trevarj
Copy link
Copy Markdown
Contributor

trevarj commented May 15, 2026

If you'd like your acks to go into the merge commit @trevarj you can use ACK <commit hash of tip>. Andrew's merge scripts will then pick up that post and add it to the merge commit. Then you can be git blamed for letting bugs past review and fun stuff like that ...

Lol, in that case I just had a tickle in my throat...

ACK 2a20232

Copy link
Copy Markdown
Member

@apoelstra apoelstra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK 2a20232; successfully ran local tests

@apoelstra apoelstra merged commit a7adaec into rust-bitcoin:master May 20, 2026
27 checks 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.

3 participants