Formatting: Further refinement to dts-linter #100247
Open
+97
−28
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
zephyrbot
requested review from
EmilioCBen,
avisconti,
dbaluta,
decsny,
duynguyenxa,
galak,
jaz1-nordic,
khoa-nguyen-18,
lstnl,
nashif,
nika-nordic,
tbursztyka,
teburd,
ttmut and
yangbolu1991
This was referenced Dec 2, 2025
kylebonnici
force-pushed
the
formatting/dts-linter-improvements
branch
2 times, most recently
from
9739668 to
ffa372f
Compare
zephyrbot
requested review from
JarmouniA,
anangl,
asmellby,
bjarki-andreasen,
danieldegrasse,
de-nordic,
fabiobaltieri,
jfischer-no and
jhedberg
zephyrbot
requested review from
JiafeiPan,
KhiemNguyenT,
MaureenHelm,
kartben,
lixuzha,
lorc and
masz-nordic
kylebonnici
force-pushed
the
formatting/dts-linter-improvements
branch
from
ffa372f to
ccc450c
Compare
Contributor
Author
|
I have been doing some testing and found some corner cases in the Linux DST files that the linter will choke on. I have fix most of these but I am not convinced we should merge this in with the DTS linter 0.3.7.... Hence I converted this PR to draft until I release the linter to address these issues. Hopefully I will release new linter later this week UPDATE Issues have been addressed and verified on Linux DTS and Zephyr files. Linter has been released with hotfix2 tag. |
The PR: - Ensure that properties have 2 new lines when node is above it. - Enures that 1 new line is required between a node and #if/#ifdef... - Enures that 2 new line are required between #endif and node. - Wraps property values that exceed 100 characters in length. These chnages need dts-linter version 0.3.7-hotfix2 dts-linter 0.3.7-hotfix2 also addresses - Update `glob` to address CVE-2025-64756 - Update `js-yaml` to address CVE-2025-64718 Signed-off-by: Kyle Bonnici <kylebonnici@hotmail.com>
kylebonnici
force-pushed
the
formatting/dts-linter-improvements
branch
from
ccc450c to
2a1afd7
Compare
|
bjarki-andreasen
approved these changes
Dec 9, 2025
tmon-nordic
approved these changes
Dec 10, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 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.
The PR:
These chnages need dts-linter version 0.3.7-hotfix2
dts-linter 0.3.7-hotfix2 also addresses
globto address CVE-2025-64756js-yamlto address CVE-2025-64718