Skip to content

treewide: fix typos to make codespell happy [backport 2023.10] #19986

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
Merged
merged 4 commits into from
Oct 19, 2023
Merged

treewide: fix typos to make codespell happy [backport 2023.10] #19986

merged 4 commits into from
Oct 19, 2023

Conversation

MrKevinWeiss
Copy link
Contributor

Backport of #19978

Contribution description

  • fixes typos in comments and docs (no generated firmware changes expected)
  • fixes a typo in a string in a GUI of a utility program
  • add some false positives to the ignore list

Testing procedure

  • No generated binaries (except for the GUI version of the utility program to flash the MSB-A2) should change
  • The diff should not look too scary

Issues/PRs references

None

maribu added 4 commits October 18, 2023 16:26
@maribu @MrKevinWeiss
boards/lsn50: change image
51069c8 
Either the wiki is down, or the image was removed from upstream. This
replaces it with the same image hosted on the zephyr doc. As a site
effect, code spell likes the new URL much more.

(cherry picked from commit 4452f07)
@maribu @MrKevinWeiss
tree-wide: fix typos in doc and comments
b104bee 
This should not change any generated binary

(cherry picked from commit edc4320)
@maribu @MrKevinWeiss
dist/tools/codespell: add more false positives
ee2525d 
(cherry picked from commit 50d5c69)
@maribu @MrKevinWeiss
dist/tools/lpc2k_pgm: fix typo in GUI / readme
9be7967 
(cherry picked from commit 50490ed)
@MrKevinWeiss MrKevinWeiss requested a review from miri64 as a code owner October 18, 2023 14:27
@MrKevinWeiss MrKevinWeiss added the Area: doc Area: Documentation label Oct 18, 2023
@MrKevinWeiss MrKevinWeiss requested a review from smlng as a code owner October 18, 2023 14:27
@MrKevinWeiss MrKevinWeiss added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Oct 18, 2023
@MrKevinWeiss MrKevinWeiss added Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer Process: release backport Integration Process: The PR is a release backport of a change previously provided to master labels Oct 18, 2023
@github-actions github-actions bot added Platform: ARM Platform: This PR/issue effects ARM-based platforms Area: network Area: Networking Area: tests Area: tests and testing framework labels Oct 18, 2023
@github-actions github-actions bot added Area: core Area: RIOT kernel. Handle PRs marked with this with care! Area: build system Area: Build system Area: pkg Area: External package ports Area: drivers Area: Device drivers Area: BLE Area: Bluetooth Low Energy support Area: tools Area: Supplementary tools Area: boards Area: Board ports Area: SAUL Area: Sensor/Actuator Uber Layer Platform: ESP Platform: This PR/issue effects ESP-based platforms Area: CoAP Area: Constrained Application Protocol implementations Area: cpu Area: CPU/MCU ports Area: USB Area: Universal Serial Bus Area: sys Area: System labels Oct 18, 2023
@riot-ci
Copy link

riot-ci commented Oct 18, 2023

Murdock results

✔️ PASSED

9be7967 dist/tools/lpc2k_pgm: fix typo in GUI / readme

Success Failures Total Runtime
7937 0 7937 17m:01s

Artifacts

@benpicco
Copy link
Contributor

bors merge

bors bot added a commit that referenced this pull request Oct 18, 2023
@bors @maribu
Merge #19986
3efdcc3 
19986: treewide: fix typos to make codespell happy [backport 2023.10] r=benpicco a=MrKevinWeiss

# Backport of #19978

### Contribution description

- fixes typos in comments and docs (no generated firmware changes expected)
- fixes a typo in a string in a GUI of a utility program
- add some false positives to the ignore list

### Testing procedure

- No generated binaries (except for the GUI version of the utility program to flash the MSB-A2) should change
- The diff should not look too scary

### Issues/PRs references

None

Co-authored-by: Marian Buschsieweke <marian.buschsieweke@posteo.net>
@bors
Copy link
Contributor

bors bot commented Oct 18, 2023

Build failed:

@maribu
Copy link
Member

maribu commented Oct 19, 2023

bors retry

native randomly misbehaving is a known issue that is more often triggered when build using LLVM. Someone(tm) really should fix native.

@bors
Copy link
Contributor

bors bot commented Oct 19, 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 921353f into RIOT-OS:2023.10-branch Oct 19, 2023
@MrKevinWeiss MrKevinWeiss deleted the backport/2023.10/fix-typos branch October 20, 2023 13:36
@MrKevinWeiss MrKevinWeiss added this to the Release 2023.10 milestone Nov 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@benpicco benpicco benpicco approved these changes

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

@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

@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

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

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

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

@DipSwitch DipSwitch Awaiting requested review from DipSwitch

@vincent-d vincent-d Awaiting requested review from vincent-d vincent-d is a code owner

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

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

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

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

Assignees
No one assigned
Labels
Area: BLE Area: Bluetooth Low Energy support Area: boards Area: Board ports Area: build system Area: Build system Area: CoAP Area: Constrained Application Protocol implementations Area: core Area: RIOT kernel. Handle PRs marked with this with care! 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: SAUL Area: Sensor/Actuator Uber Layer Area: sys Area: System Area: tests Area: tests and testing framework Area: tools Area: Supplementary tools Area: USB Area: Universal Serial Bus CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer Platform: ARM Platform: This PR/issue effects ARM-based platforms Platform: ESP Platform: This PR/issue effects ESP-based platforms Process: release backport Integration Process: The PR is a release backport of a change previously provided to master
Projects
None yet
Milestone
Release 2023.10
Development

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