Skip to content

Resolve rustfmt/changelog CI failures in STM32F469 Disco display-touch updates#16

Merged
Amperstrand merged 2 commits intomasterfrom
copilot/fix-formatting-errors
Feb 15, 2026
Merged

Resolve rustfmt/changelog CI failures in STM32F469 Disco display-touch updates#16
Amperstrand merged 2 commits intomasterfrom
copilot/fix-formatting-errors

Conversation

Copy link

Copilot AI commented Feb 15, 2026

The PR fixes the formatting regressions reported by the rustfmt job (including macro-call layout and assignment handling in touch error throttling paths) and adds the missing changelog update required by changelog-enforcer.

  • Rustfmt conformance in touch polling paths

    • Normalized defmt::warn! call formatting in examples/f469disco-lcd-test.rs to rustfmt-accepted form.
    • Kept explicit counter mutation semantics in retry/error-throttle code paths while aligning with formatter output.
  • Whitespace/format cleanup in example module

    • Removed formatting-only drift in examples/f469disco/nt35510.rs (trailing-space delta picked up by rustfmt).
  • Changelog gate fix

    • Added an Unreleased entry in CHANGELOG.md summarizing the PR scope (STM32F469 Disco display/touch example improvements and dma module exposure).
// before (non-rustfmt form in failing diff)
defmt::warn!(
    "detect_touch read error (attempt {})",
    attempt + 1
);

// after
defmt::warn!("detect_touch read error (attempt {})", attempt + 1);

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: Amperstrand <141745238+Amperstrand@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix formatting errors in multiline macro calls and assignments Resolve rustfmt/changelog CI failures in STM32F469 Disco display-touch updates Feb 15, 2026
Copilot AI requested a review from Amperstrand February 15, 2026 05:26
@Amperstrand Amperstrand marked this pull request as ready for review February 15, 2026 07:49
@Amperstrand Amperstrand merged commit 88e8c28 into master Feb 15, 2026
22 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.

2 participants