Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

"Fix" clippy checks in CI, resolve a bunch of clippy warnings #1134

Merged
merged 8 commits into from
Feb 2, 2024
Merged

"Fix" clippy checks in CI, resolve a bunch of clippy warnings #1134

merged 8 commits into from
Feb 2, 2024

Commits on Feb 2, 2024

  1. Fix a bunch of clippy warnings in the esp-hal build script

    @jessebraham
    jessebraham committed Feb 2, 2024
    Configuration menu
    Copy the full SHA
    f088c18 View commit details
    Browse the repository at this point in the history

  2. Resolve most clippy warnings for ESP32 (no features)

    @jessebraham
    jessebraham committed Feb 2, 2024
    Configuration menu
    Copy the full SHA
    d77275a View commit details
    Browse the repository at this point in the history

  3. Do the same for ESP32-S2/S3...

    @jessebraham
    jessebraham committed Feb 2, 2024
    Configuration menu
    Copy the full SHA
    10dd118 View commit details
    Browse the repository at this point in the history

  4. Same for C2/C3...

    @jessebraham
    jessebraham committed Feb 2, 2024
    Configuration menu
    Copy the full SHA
    7c825e2 View commit details
    Browse the repository at this point in the history

  5. CI check for clippy now actually works (and passes!) for Xtensa

    @jessebraham
    jessebraham committed Feb 2, 2024
    Configuration menu
    Copy the full SHA
    4d648d6 View commit details
    Browse the repository at this point in the history

  6. RISC-V chips (except P4) now pass clippy in CI

    @jessebraham
    jessebraham committed Feb 2, 2024
    Configuration menu
    Copy the full SHA
    448f80c View commit details
    Browse the repository at this point in the history

  7. Add some safety doc comments which I forgot to write

    @jessebraham
    jessebraham committed Feb 2, 2024
    Configuration menu
    Copy the full SHA
    154a136 View commit details
    Browse the repository at this point in the history

  8. The PwmPeripheral trait is no longer unsafe, and is instead now s… 

    …ealed
    @jessebraham
    jessebraham committed Feb 2, 2024
    Configuration menu
    Copy the full SHA
    a07be02 View commit details
    Browse the repository at this point in the history