-
Notifications
You must be signed in to change notification settings - Fork 144
Conversation
'beginCaptures': | ||
'0': | ||
'name': 'punctuation.definition.string.begin.ruby' | ||
'contentName': 'source.sql' |
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.
source.graphql
'include': '#interpolated_ruby' | ||
} | ||
{ | ||
'include': 'source.sql' |
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 do believe you meant source.graphql here?
8a2c4f9
to
063fe44
Compare
😆 Sorry about that. I was having a hard time testing out the changes locally, and I ended up with two different copies of the source. Apparently I pushed the wrong one! Force-pushed with corrections. |
Thanks! |
Hi, is there a release schedule for this project? I'm looking forward to this feature but I'm happy to wait if a release is planned sometime in the future. |
It's currently planned to be included in the Atom 1.15.0 release. We don't publish release dates, however. |
Thanks for the info, I'll keep an eye out for that Atom release! |
Requirements
Description of the Change
I recently discovered that language-ruby will highlight SQL inside
<<-SQL
heredoc blocks. Cool!I often use
<<-GRAPHQL
to write inline GraphQL queries, so I thought, maybe I can modify language-ruby to highlight GraphQL within those blocks.This PR adds GraphQL highlighting within
<<-GRAPHQL
heredoc blocks, as shown above.Alternate Designs
I didn't consider alternate designs, I just copied and modified one of the other heredoc highlighters.
Benefits
GraphQL users can have highlighted GraphQL inside their Ruby files.
Possible Drawbacks
This introduces a maintenance burden: more lines of code means more things that can break 😬
This heredoc highlighter is not tested. I didn't find tests for the others, either. Are there any?
Applicable Issues
Fixes #173
Thanks for considering this change!