-
Notifications
You must be signed in to change notification settings - Fork 7k
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
doc: sphinx: Add Pygment lexer for DTS #6397
Conversation
Add pygment lexer for DTS code-block. Fixes zephyrproject-rtos#6029 Origin: skiboot, https://github.com/qemu/skiboot Status: 91350c5a926795d8917a4eff699941361f780476 Description: Syntax highlighting of Device Tree Source (dts) Dependencies: None URL: https://github.com/qemu/skiboot/blob/master/doc/DtsLexer.py commit: 91350c5a926795d8917a4eff699941361f780476 Maintained-by: External License: BSL-1.0 License Link: https://github.com/qemu/skiboot/blob/master/doc/DtsLexer.py Signed-off-by: Bobby Noelte <b0661n0e17e@gmail.com>
Codecov Report
@@ Coverage Diff @@
## master #6397 +/- ##
=======================================
Coverage 53.08% 53.08%
=======================================
Files 430 430
Lines 41040 41040
Branches 7908 7908
=======================================
Hits 21785 21785
Misses 16032 16032
Partials 3223 3223 Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
While I don't object to adding DTS support, @nashif hasn't been keen on pre-empting delayed upstream patches by adding them to the Zephyr project directly. Looks like this code came from here: https://patchwork.ozlabs.org/patch/688187/. Will we need to add this BSD licensed (?) item to our licensing list? I'll defer to Anas for his approval.
The reference is to skiboot where this patch stems from (and is applied). For delayed upstream one would have to look at pygments. I haven´t found the pull request there https://bitbucket.org/birkenfeld/pygments-main/pull-request. |
thanks for clarifying this hasn't been submitted upstream to pygments. |
@nashif is there anything open that prevents a merge? Thx Bobby |
Add pygment lexer for DTS code-block.
Fixes #6029
Originally developed for #5799.
Signed-off-by: Bobby Noelte b0661n0e17e@gmail.com