ambiot / ambd_arduino

AmebaD Arduino third-party package SDK
MIT License
81 stars 51 forks source link

Spelling mistakes (GitHub Action Spell Check) #218

Closed designer2k2 closed 4 months ago

designer2k2 commented 4 months ago

Version

latest dev (checkout manually)

Description

There are multiple spelling mistakes in this repo, mostly typos like: ./Arduino_package/hardware/cores/ambd/wifi_drv.cpp:18: modifed ==> modified

I would purpose a GitHub action "spell check" to automatically check the repo and list all. This will then also check all future commits before they get merged. Is there a interest into something like this here?

The spell check action: https://github.com/designer2k2/ambd_arduino/blob/designer2k2-patch-1/.github/workflows/spell-check.yml

Here is a example on how it runs: https://github.com/designer2k2/ambd_arduino/actions/runs/7846123899/job/21412100274#step:4:17 (this run reports about 1800 errors)

I can make a pull request with this action if wanted.

M-ichae-l commented 4 months ago

@designer2k2 Thanks for pointed it out! Please refer to this commit #84a9d07235c2db534fc9cb3fddad09bada7ed6ad Under Arduino_package/hardware

designer2k2 commented 4 months ago

Hello @M-ichae-l. the commit is a good start! Great to see it :-)

Some points from my side i would consider:

This currently results in 3 errors (2x Celsius, 1x below)

Further i suggest to remove the filter for specific files, that results in some other changes needed:

The fully changed spell_check.yml: https://github.com/designer2k2/ambd_arduino/blob/designer2k2-patch-1/.github/workflows/spell_check.yml

And a run with the modifications: https://github.com/designer2k2/ambd_arduino/actions/runs/7891851038/job/21537043921

M-ichae-l commented 4 months ago

@designer2k2 for "ser" and "hart" are from third party files.

designer2k2 commented 4 months ago

Thanks! https://github.com/ambiot/ambd_arduino/commit/e8df7125f9f0701eea4e57cb6fa07526cd1ced72 is closing this here 👍