-
-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
feat: Add support for nginx configuration files #11823
base: master
Are you sure you want to change the base?
Conversation
0b49850
to
1b30dd6
Compare
1b30dd6
to
a01338a
Compare
name = "nginx" | ||
scope = "source.nginx" | ||
injection-regex = "nginx" | ||
file-types = ["conf"] |
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.
This will conflict with hocon
. We might need a glob pattern that would match the common places to put nginx files instead
roots = [] | ||
comment-token = "#" | ||
indent = { tab-width = 4, unit = " " } | ||
auto-format = false |
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.
roots = [] | |
comment-token = "#" | |
indent = { tab-width = 4, unit = " " } | |
auto-format = false | |
comment-token = "#" | |
indent = { tab-width = 4, unit = " " } |
these keys are set to their defaults so they can be removed
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.
The capture names here need to be adjusted to the ones we use: https://docs.helix-editor.com/master/themes.html#syntax-highlighting
For example @number
should become @constant.numeric
(number) @number | ||
(metric) @number | ||
|
||
(regex) @regex |
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.
Let's highlight this as a string
and add an injection for it in injections.scm to use the regex language:
((regex) @injection.content
(#set! injection.language "regex"))
No description provided.