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

Hide "TODO" from grammar-selector menu #65

Merged
merged 2 commits into from Sep 25, 2017

Conversation

Projects
None yet
2 participants
@Alhadis
Contributor

Alhadis commented Sep 25, 2017

Description of the Change

Follow-up of atom/grammar-selector#34, which patched the grammar-selector package to hide "internal" or injection-specific grammars from the selection menu. This was recently merged and released in grammar-selector@0.49.6. It's now safe to cut redundant name fields from grammars which shouldn't be listed alongside "real" languages.

See atom/grammar-selector#34 for the original discussion.

/cc @50Wliu

Alhadis added some commits Sep 25, 2017

Remove name field from grammar
The `name` property of language grammars is only used to provide a human
readable identifier in GUIs. For "injection-only" grammars such as this,
it's not only redundant, but confusing for users who happen across it in
the grammar-selector.

References: atom/grammar-selector#34
Delete redundant `.coffeelintignore` file
There is no `spec/fixtures` directory, and there never was either.

Related: atom/language-javascript#464

@50Wliu 50Wliu merged commit 8300d78 into atom:master Sep 25, 2017

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment