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.

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

Allow clippy to break API, ignore a few initialism errors #2840

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

bugadani
Copy link
Contributor

@bugadani bugadani commented Dec 19, 2024

Since esp-hal is not yet stable, we can enable clippy lints that would need API-breaking changes to fix. This allows us to detect potential existing issues (marked with a FIXME, or corrected in this PR), as well as future problems that we might introduce by accident.

We can keep this mechanism enabled even after the first stable release - it will prevent introducing future changes. We will need to be mindful of new lints added to this category after stabilization, though, to avoid breaking our API.

The list of issues detected isn't very large and it mostly considers enum variant naming.

skip-changelog as I created the clippy config but didn't actually make code changes to some crates.

@bugadani bugadani force-pushed the clippy branch 7 times, most recently from 17ca948 to 0400a3f Compare December 20, 2024 11:07
@bugadani bugadani added the skip-changelog No changelog modification needed label Dec 20, 2024
@bugadani bugadani marked this pull request as ready for review December 20, 2024 11:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip-changelog No changelog modification needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant