Pug/Jade Syntax
AbandonedPublic

Authored by guoyunhe on Apr 15 2017, 10:45 AM.

Details

Reviewers
None
Group Reviewers
Framework: Syntax Highlighting
KTextEditor
Kate
Summary

Pug, or Jade, is a very popular template engine for NodeJS applications. This syntax file supports most Pug syntax.

https://pugjs.org/api/getting-started.html

Test Plan

Diff Detail

Repository
R216 Syntax Highlighting
Lint
Lint Skipped
Unit
Unit Tests Skipped
guoyunhe created this revision.Apr 15 2017, 10:45 AM
Restricted Application added a project: Frameworks. · View Herald TranscriptApr 15 2017, 10:45 AM
Restricted Application added a subscriber: Frameworks. · View Herald Transcript
guoyunhe edited the summary of this revision. (Show Details)Apr 15 2017, 10:46 AM
guoyunhe edited the summary of this revision. (Show Details)

Do you also agree to release this highlighting file under MIT license? Same for the test case you attached? Also, can we use this test case for unit testing?

Do you also agree to release this highlighting file under MIT license? Same for the test case you attached? Also, can we use this test case for unit testing?

It is okay. I can release the syntax file under LGPL, MIT, BSD ( https://community.kde.org/Policies/Licensing_Policy )

The test file is copied from pugjs.org , I am not the author. It is MIT licensed https://github.com/pugjs/pug-www/blob/39257987117795a4c2d133618649bb89f1cc81e4/package.json

I found this part might be a bug in this syntax highlight:

mimetype="text/plain"

It will highlight all plain text file to pug syntax. Should I just change mimetype to empty? Not sure how to write it for files that don't have registered mimetype.

yep, keep the mimetype field empty in that case

guoyunhe updated this revision to Diff 13734.Apr 24 2017, 11:55 AM

Remove mimetype

Could someone approve this patch? So I can commit it.

guoyunhe abandoned this revision.Aug 23 2017, 8:06 PM