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

Update CI workflow for Arduino project-specific linting #44

Merged
merged 1 commit into from
Apr 21, 2021
Merged

Update CI workflow for Arduino project-specific linting #44

merged 1 commit into from
Apr 21, 2021

Conversation

per1234
Copy link
Contributor

@per1234 per1234 commented Apr 20, 2021

On every push, pull request, and periodically, run Arduino Lint to check for common problems not related to the project code.

This replaces the "Extra Library Checks" workflow in all but one respect: keywords.txt linting. Arduino Lint does not currently do any validation of the keywords.txt data, so use of the per1234/arduino-ci-script script is still required for this specific check. A dedicated "Check keywords.txt" workflow was added for that purpose.

Fixes #42

On every push, pull request, and periodically, run Arduino Lint to check for common problems not related to the project
code.

This replaces the "Extra Library Checks" workflow in all but one respect: keywords.txt linting. Arduino Lint does not
currently do any validation of the keywords.txt data, so use of the `per1234/arduino-ci-script` script is still required
for this specific check. A dedicated "Check keywords.txt" workflow was added for that purpose.
@per1234 per1234 added type: enhancement PR to improve the project. topic: ci Continuous integration for this repository. labels Apr 20, 2021
@aentinger aentinger merged commit 740f474 into 107-systems:master Apr 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: ci Continuous integration for this repository. type: enhancement PR to improve the project.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add arduino-lint GitHub Action
2 participants