ci: add compliance script checks to example repo#101
Open
danieldegrasse wants to merge 3 commits into
Open
Conversation
Add configuration files for linters used by Zephyr compliance checks. These files are duplicated from the Zephyr repo. Signed-off-by: Daniel DeGrasse <daniel.degrasse@analog.com>
Format example_west_command to comply with ruff CI checks Signed-off-by: Daniel DeGrasse <daniel.degrasse@analog.com>
danieldegrasse
force-pushed
the
feature/compliance-script
branch
from
f750b68 to
bfaded7
Compare
danieldegrasse
commented
Jul 3, 2026
Contributor
Author
There was a problem hiding this comment.
Compliance checks are failing due to extra newlines in this file, but I'm not sure if we want to strip those? I assume this license file was pulled directly from apache.org
Contributor
There was a problem hiding this comment.
IMHO better not to strip. Messing with the defaults of what REUSE fetches will cause extra work when doing FOSS compliance.
Also, having an example on how to create an exception would be valuable too.
Add support for running compliance checks on code contributed to the example repository, as a demonstration for how to implement custom compliance checks on top of those Zephyr provides Signed-off-by: Daniel DeGrasse <daniel.degrasse@analog.com>
danieldegrasse
force-pushed
the
feature/compliance-script
branch
from
bfaded7 to
872fa10
Compare
| run: | | ||
| if ! command -v west &> /dev/null; then | ||
| echo "west could not be found, installing..." | ||
| python -m pip install west==1.5.0 |
Contributor
There was a problem hiding this comment.
Nit: To give a good example, this better be fixed to a cryptographically secure hash.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add CI job to run compliance script checks within the example repo, as a demonstration of how to utilize and build on Zephyr's compliance checks.
This also enforces the compliance checks on this repo, so we need to reformat a few of the python scripts to comply with them