Skip to content

Pin compatible nightly rustfmt - #1120

Merged
tnull merged 1 commit into
lightningdevkit:mainfrom
tnull:2026-09-pin-compatible-rustfmt
Oct 12, 2026
Merged

tnull merged 1 commit into
lightningdevkit:mainfrom
tnull:2026-09-pin-compatible-rustfmt

Conversation

@tnull

@tnull tnull commented Sep 28, 2026

Copy link
Copy Markdown
Collaborator

Nightly rustfmt from September 23 rewrites code that stable rustfmt changes back, causing the weekly formatting PR to fail CI. Pin the last compatible nightly to keep weekly formatting usable.

Co-Authored-By: HAL 9000

Nightly rustfmt from September 23 rewrites code that stable rustfmt
changes back, causing the weekly formatting PR to fail CI. Pin the
last compatible nightly to keep weekly formatting usable.

Co-Authored-By: HAL 9000
@ldk-reviews-bot

ldk-reviews-bot commented Sep 28, 2026 •

Copy link
Copy Markdown

👋 I see @TheBlueMatt was un-assigned.
If you'd like another reviewer assignment, please click here.

@tnull

tnull commented Oct 12, 2026

Copy link
Copy Markdown
Collaborator Author

Reassigned to @joostjager to get this out of the way soon, it's trivial.

@tnull
tnull removed the request for review from TheBlueMatt October 12, 2026 07:27

@joostjager joostjager left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is it useful to track switching back to latest nightly? (I think when stable rust changes?)

@tnull
tnull merged commit 66f21e0 into lightningdevkit:main Oct 12, 2026
23 of 25 checks passed
@tnull

tnull commented Oct 12, 2026

Copy link
Copy Markdown
Collaborator Author

Is it useful to track switching back to latest nightly? (I think when stable rust changes?)

Hmm, I don't know, yes eventually we should bump it, but not sure it's worth tracking anywhere separately.

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