Skip to content
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

Support for JavaScript tagged template literals #8

Merged
merged 4 commits into from Sep 3, 2017
Merged

Conversation

jparise
Copy link
Owner

@jparise jparise commented Sep 3, 2017

Building on #1, this commit makes the following minor changes:

  1. g:graphql_tag_names => g:graphql_javascript_tags
  2. jsTemplateVar => jsTemplateExpression, to reflect the change made in pangloss/vim-javascript@88c85d9
  3. Added Relay.QL to the list of recognized JavaScript tags
  4. Unit test support

tessro and others added 4 commits October 19, 2016 21:45
This patch adds support for GraphQL syntax in ES6 tagged template
literals. (As used by `graphql-tag`, for example.) By default, GraphQL
syntax is highlighted in template literals tagged with `gql` or
`graphql`. This can be configured via the `g:graphql_tag_names` array.
…ia/vim-graphql into paulrosania-tagged-template-literals
Building on #1, this commit makes the following minor changes:

 1. `g:graphql_tag_names` => `g:graphql_javascript_tags`
 2. `jsTemplateVar` => `jsTemplateExpression`, to reflect the
    change made in pangloss/vim-javascript@88c85d9
 3. Added `Relay.QL` to the list of recognized JavaScript tags
@jparise jparise changed the title Revised support for JavaScript tagged literals Full support for JavaScript tagged literals Sep 3, 2017
@jparise jparise changed the title Full support for JavaScript tagged literals Support for JavaScript tagged literals Sep 3, 2017
@jparise jparise changed the title Support for JavaScript tagged literals Support for JavaScript tagged template literals Sep 3, 2017
@jparise jparise merged commit 84e3748 into master Sep 3, 2017
@jparise jparise deleted the js-literals branch September 3, 2017 16:28
@jparise jparise mentioned this pull request Oct 9, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants