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

Use GitHub Actions for continuous integration #2

Merged
merged 1 commit into from
Dec 16, 2019
Merged

Use GitHub Actions for continuous integration #2

merged 1 commit into from
Dec 16, 2019

Conversation

per1234
Copy link
Contributor

@per1234 per1234 commented Dec 16, 2019

On every push and pull request, all the example sketches will be compiled for MKR Zero and all files in the repository will be checked for misspelled words.

The spell-check action found some misspelled words in the spiffs* files. Since this is an external dependency, fixing the true positives among these in this repository would lead to difficulty merging subsequent releases of spiffs. So I have added the true and false positives to extras/codespell-ignore-words-list.txt. I have submitted a PR upstream to fix the true positives: pellepl/spiffs#254. Once that is merged and pulled into this repo, the true positives may be removed from the list to avoid an increased possibility of false negatives.

@per1234
Copy link
Contributor Author

per1234 commented Dec 16, 2019

It looks like GitHub won't run the actions submitted in a PR. You can see the successful run from my fork here:
https://github.com/per1234/Arduino_MKRMEM/runs/350302661

@aentinger
Copy link
Collaborator

@per1234 Thank you very much for adding CI to this repository. I have only one small request - can you please extend the fqbn list to incorporate all MKR family boards?

On every push and pull request, all the example sketches will be compiled for MKR Zero and all files in the repository will be checked for misspelled words.

The spell-check action found some misspelled words in the spiffs* files. Since this is an external dependency, fixing the true positives among these in this repository would lead to difficulty merging subsequent releases of spiffs. So I have added the true and false positives to extras/codespell-ignore-words-list.txt. I have submitted a PR upstream to fix the true positives. Once that is merged and pulled into this repo, the true positives may be removed from the list to avoid an increased possibility of false negatives.
@aentinger aentinger merged commit 913ae6d into arduino-libraries:master Dec 16, 2019
@per1234 per1234 deleted the CI branch December 16, 2019 12:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants