-
Notifications
You must be signed in to change notification settings - Fork 1
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
fixed define
command
#13
Conversation
* added `dont-repeat` command * tests: * renamed "document.gdb" to command".gdb * test.gdb: moved entries to hook.gdb / command.gdb
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.
I prefer to have both define.gdb
& document.gdb
, even if document
use a commandname
from command
.
Also reading the doc, define & define-prefix are different in what they do, so we have to separate them especially since i didn't grasp the differences between them
@@ -214,6 +214,13 @@ | |||
"2": { "name": "constant.numeric.count.ignore.gdb" } | |||
} | |||
}, | |||
{ | |||
"name": "meta.command.dont.gdb", |
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.
I prefer to use the full command name for "name" field, to avoid ambiguities so meta.command.dont-repeat.gdb
@@ -774,7 +781,7 @@ | |||
}, | |||
{ | |||
"name": "meta.block.define-command.gdb", | |||
"begin": "^[ \t]*(define(?:-prefix)?\\b[ \t]+comm(?:a(?:n(?:d)?)?)?)[ \t]+([^[ \t]]+)[ \t]*$", | |||
"begin": "^[ \t]*(define(?:-prefix)?)[ \t]+([^[ \t]]+)[ \t]*$", |
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.
@pirpyn Would you mind taking this over? |
Sure, likely tomorrow |
For some unknown reason I've added that wrong, there's no "command" in the define command.
Change details:
define
commanddont-repeat
(commonly only used indefine
, but not raising an error outside either)