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

treewide: fix typos and false positives found by codespell [backport 2023.04] #19563

Merged
merged 3 commits into from
May 10, 2023
Merged

treewide: fix typos and false positives found by codespell [backport 2023.04] #19563

merged 3 commits into from
May 10, 2023

Conversation

MrKevinWeiss
Copy link
Contributor

Backport of #19528

Contribution description

This fixes some typos and adds one correctly used abbreviation to the ignore list.

Testing procedure

If bors merges this without static tests failing, it did what it intended to do.

Issues/PRs references

Fixes issues that prevent #19440 from merging

maribu added 3 commits May 10, 2023 08:28
@maribu @MrKevinWeiss
tree wide: fix typos in comments found by codespell
2e62889 
This will not change generated binaries, only the Doxygen output and
source code will have less typos.

(cherry picked from commit c3500eb)
@maribu @MrKevinWeiss
sys/ztimer: fix typo in debug output
6648b34 
(cherry picked from commit 8fbbc40)
@maribu @MrKevinWeiss
dist/tools/codespell: ignore false positives
a3e77f6 
Add two false positives to the list of words to ignore.

(cherry picked from commit 73e7e30)
@MrKevinWeiss MrKevinWeiss added the Area: boards Area: Board ports label May 10, 2023
@MrKevinWeiss MrKevinWeiss requested a review from fjmolinas as a code owner May 10, 2023 06:28
@MrKevinWeiss MrKevinWeiss added Area: cpu Area: CPU/MCU ports Area: doc Area: Documentation labels May 10, 2023
@MrKevinWeiss MrKevinWeiss requested a review from kaspar030 as a code owner May 10, 2023 06:28
@MrKevinWeiss MrKevinWeiss added Area: drivers Area: Device drivers Area: network Area: Networking labels May 10, 2023
@MrKevinWeiss MrKevinWeiss requested a review from bergzand as a code owner May 10, 2023 06:28
@MrKevinWeiss MrKevinWeiss added the Area: pkg Area: External package ports label May 10, 2023
@MrKevinWeiss MrKevinWeiss requested a review from miri64 as a code owner May 10, 2023 06:28
@MrKevinWeiss MrKevinWeiss added Area: sys Area: System Area: tests Area: tests and testing framework labels May 10, 2023
@MrKevinWeiss MrKevinWeiss added Area: timers Area: timer subsystems Area: tools Area: Supplementary tools CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels May 10, 2023
@MrKevinWeiss MrKevinWeiss added Platform: ARM Platform: This PR/issue effects ARM-based platforms Platform: native Platform: This PR/issue effects the native platform labels May 10, 2023
@MrKevinWeiss MrKevinWeiss requested a review from cgundogan as a code owner May 10, 2023 06:28
@MrKevinWeiss MrKevinWeiss added the Process: release backport Integration Process: The PR is a release backport of a change previously provided to master label May 10, 2023
@MrKevinWeiss MrKevinWeiss requested a review from basilfx as a code owner May 10, 2023 06:28
@MrKevinWeiss MrKevinWeiss mentioned this pull request May 10, 2023
Merged
@riot-ci
Copy link

riot-ci commented May 10, 2023

Murdock results

✔️ PASSED

a3e77f6 dist/tools/codespell: ignore false positives

Success Failures Total Runtime
6882 0 6882 11m:12s

Artifacts

@MrKevinWeiss
Copy link
Contributor Author

Seems like half the maintainers of riot were requested to review this one...

@maribu
Copy link
Member

maribu commented May 10, 2023

bors merge

@bors
Copy link
Contributor

bors bot commented May 10, 2023

Build succeeded!

The publicly hosted instance of bors-ng is deprecated and will go away soon.

If you want to self-host your own instance, instructions are here.
For more help, visit the forum.

If you want to switch to GitHub's built-in merge queue, visit their help page.

@bors bors bot merged commit 89fc055 into RIOT-OS:2023.04-branch May 10, 2023
@MrKevinWeiss MrKevinWeiss deleted the backport/2023.04/dist/tools/codespell branch May 12, 2023 13:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@maribu maribu maribu approved these changes

@smlng smlng Awaiting requested review from smlng

@leandrolanzieri leandrolanzieri Awaiting requested review from leandrolanzieri leandrolanzieri is a code owner

@aabadie aabadie Awaiting requested review from aabadie aabadie is a code owner

@MichelRottleuthner MichelRottleuthner Awaiting requested review from MichelRottleuthner MichelRottleuthner is a code owner

@fjmolinas fjmolinas Awaiting requested review from fjmolinas fjmolinas is a code owner

@kaspar030 kaspar030 Awaiting requested review from kaspar030 kaspar030 is a code owner

@bergzand bergzand Awaiting requested review from bergzand bergzand is a code owner

@miri64 miri64 Awaiting requested review from miri64 miri64 is a code owner

@haukepetersen haukepetersen Awaiting requested review from haukepetersen haukepetersen is a code owner

@PeterKietzmann PeterKietzmann Awaiting requested review from PeterKietzmann PeterKietzmann is a code owner

@cgundogan cgundogan Awaiting requested review from cgundogan cgundogan is a code owner

@jeandudey jeandudey Awaiting requested review from jeandudey jeandudey is a code owner

@basilfx basilfx Awaiting requested review from basilfx basilfx is a code owner

Assignees
No one assigned
Labels
Area: boards Area: Board ports Area: cpu Area: CPU/MCU ports Area: doc Area: Documentation Area: drivers Area: Device drivers Area: network Area: Networking Area: pkg Area: External package ports Area: sys Area: System Area: tests Area: tests and testing framework Area: timers Area: timer subsystems Area: tools Area: Supplementary tools CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Platform: ARM Platform: This PR/issue effects ARM-based platforms Platform: native Platform: This PR/issue effects the native platform Process: release backport Integration Process: The PR is a release backport of a change previously provided to master
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@MrKevinWeiss @riot-ci @maribu