Details
- Reviewers
dhaumann - Group Reviewers
Framework: Syntax Highlighting - Commits
- R216:bbb5188d9e12: Add more highlighting for nested languages in markdown.
Diff Detail
- Repository
- R216 Syntax Highlighting
- Branch
- nested (branched from master)
- Lint
No Linters Available - Unit
No Unit Test Coverage - Build Status
Buildable 8766 Build 8784: arc lint + arc unit
Could you extend autotests/input/test.markdown ?
Then call ./autotests/update-reference-data.sh in the build folder to copy the changes into the src folder, and update this patch.
PS: And increase the "version" in the langauge xml element at the beginning of the file :-)
Even folding works inside fenced blocks.
After running ./autotests/update-reference-data.sh autotests/folding/example.rmd.fold changed. I did not commit that change.
The language version is now updated and I added a few more languages.
Looks good to me, thanks.
Before committing, could you increase the kateversion="5.0" in the language xml element as well? Otherwise KDE 4 users could get this via the download dialog as well, without getting the depending files due to 5.0 requirements in the other xml files. Then, the local xml files are inconsistent and broken.
I see that one test fails after committing this patch. The example output was modified by /autotests/update-reference-data.sh but I did not submit it. I thought it was not related to the markdown change.
Hi Jos, could you please check again the output of the highlighting indexer?
ContextChecker::check: "/home/dh/kde/kf5/src/frameworks/syntax-highlighting/data/syntax/markdown.xml" Unused contexts: QSet("kconfig-code")
The checker says that this context is unused... I think this is a bug.
Jos fixed it in commit
1653403267ed9ad0a27ee4042be8e0d714a5dcc4
I made that check now required.
All current files pass, people can comment out contexts during development.