CI: Fix failure #135
Open
CI: Fix failure #135
Conversation
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
soburi
changed the title
soburi
force-pushed
the
variant_name
branch
14 times, most recently
from
67ea236 to
194d31a
Compare
soburi
requested review from
DhruvaG2000,
szczys and
beriberikix
as code owners
soburi
force-pushed
the
variant_name
branch
12 times, most recently
from
c960140 to
584d40f
Compare
soburi
force-pushed
the
variant_name
branch
11 times, most recently
from
17f4c2e to
3fe9c4a
Compare
soburi
changed the title
soburi
changed the title
Use the checkpatch.pl included with Zephyr to perform the check. Signed-off-by: TOKITA Hiroshi <[email protected]>
Changed boards to be specified by their normalized names. Signed-off-by: TOKITA Hiroshi <[email protected]>
Modify to use west.yml included in the PR version. Accordingly, the initialization process has been changed. Signed-off-by: TOKITA Hiroshi <[email protected]>
Added cmsis_6 to the repository in line with the update of Zephyr. Signed-off-by: TOKITA Hiroshi <[email protected]>
Apply newest zephyr version Signed-off-by: TOKITA Hiroshi <[email protected]>
Get the variant name (NORMALIZED_BOARD_TARGET) from the Zephyr build system as early as possible. This allows to have per-target build directories. Note that a bug with the shield specifiers does not currently allow to reuse the same build directory multiple times. Signed-off-by: TOKITA Hiroshi <[email protected]> Co-authored-by: Luca Burelli <[email protected]>
|
Could you take a look, please? |
Ayush1325
reviewed
Aug 25, 2025
There was a problem hiding this comment.
Can you explain why the changes to license_check are required?
There was a problem hiding this comment.
The github action is stuck with the current version.
It was originally brought over from the Zephyr main body, so I applied the current one.
Updated commit message.
Ayush1325
approved these changes
Aug 26, 2025
soburi
force-pushed
the
variant_name
branch
2 times, most recently
from
484a5ea to
eb778d1
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
CI is failing for several reasons.
We will fix these.
Also, west.yml references the main repository,
not the one in the PR, so we will change it to use the one in the PR.
This workflow uses the Linux version of
checkpatch.pl.We change it to zephyr's version.