search

Digitalbrainstem / ejs-grammar

EJS Grammar for VSCode
MIT License
75 stars 28 forks source link

`scripts` entry in package.json #13

Closed meesfrensel closed 5 years ago

meesfrensel commented 6 years ago

There is an entry in scripts in package.json:

"scripts": {
    "update-grammar": "node ../../build/npm/update-grammar.js grammars/ejs.json ./syntaxes/ejs.json"
}

However, update-grammar is not a standard script, at least it's not listed in the official docs (which list one script, and point to the npm docs for other standard npm scripts, where it's also not listed). I therefore think it should be removed entirely. Of course, unless there is a purpose that I can't find!

ExE-Boss commented 5 years ago

This is run using npm run update-grammar.

As for its purpose, the author of this package uses it to update the ejs.json TextMate language definition.

It will become obsolete by https://github.com/Digitalbrainstem/ejs-grammar/pull/23, which completely rewrites the language definition.