Skip to content

Commit 92d526e

Browse files
committed
Merge remote-tracking branch 'origin/master' into v2
# Conflicts: # resources/META-INF/plugin.xml # src/main/com/intellij/lang/jsgraphql/ide/highlighting/JSGraphQLSyntaxHighlighter.java # src/main/com/intellij/lang/jsgraphql/ide/notifications/JSGraphQLNodeInterpreterEditorNotificationProvider.java # src/main/com/intellij/lang/jsgraphql/schema/ide/search/JSGraphQLDefinitionsSearchExecutor.java # src/main/com/intellij/lang/jsgraphql/v1/ide/project/JSGraphQLLanguageUIProjectService.java
2 parents bf2db2f + b9a6b73 commit 92d526e

File tree

5 files changed

+66
-3
lines changed

5 files changed

+66
-3
lines changed

.github/ISSUE_TEMPLATE.md

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,17 @@
1+
## Before you create the issue
2+
- Please search the existing issues at https://github.com/jimkyndemeyer/js-graphql-intellij-plugin/issues?q=sort%3Aupdated-desc before opening a new issue.
3+
- Read the FAQ at https://github.com/jimkyndemeyer/js-graphql-intellij-plugin#faq
4+
- If you're using an EAP, try with a stable version. If that works, please mention it in the issue along with both versions.
5+
6+
## Version and Environment Details
7+
__Operation system__:
8+
9+
__IDE name and version__:
10+
11+
__Plugin version__:
12+
13+
## Expected Behaviour
14+
15+
## Actual Behaviour
16+
17+
## Steps to Reproduce / Link to Repo with Reproduction and Instructions

CHANGELOG.md

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,12 @@
1+
## 1.7.3 (2019-01-26)
2+
3+
Fixes:
4+
5+
- Fixed missing syntax highlighting colors due to uninitialized JavaScript colors being referenced from GraphQL (#158)
6+
- Set default charset to UTF-8 in query result viewer to align with JSON spec (#181)
7+
- Query result viewer doesn't always reformat the response JSON (#209)
8+
- Make the Query result viewer work with windows line endings in JSON responses (#191)
9+
110
## 1.7.2 (2018-05-29)
211

312
Fixes:

README.md

Lines changed: 37 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,8 @@
22

33
# JS GraphQL IntelliJ Plugin
44

5+
__Note:__ V2.0 is in active development. See the [v2 branch](https://github.com/jimkyndemeyer/js-graphql-intellij-plugin/tree/v2) and [#164]( https://github.com/jimkyndemeyer/js-graphql-intellij-plugin/issues/164)
6+
57
GraphQL language support including Relay.QL tagged templates in JavaScript and TypeScript.
68

79
It provides the following features in IntelliJ IDEA, WebStorm, RubyMine, PhpStorm, and PyCharm:
@@ -14,7 +16,41 @@ It provides the following features in IntelliJ IDEA, WebStorm, RubyMine, PhpStor
1416
- Configurable GraphQL schema retrieval and reloading based on a local file or a url using 'then-request'
1517
- Execute queries with variables against configurable endpoints
1618

17-
It depends on [js-graphql-language-service](https://github.com/jimkyndemeyer/js-graphql-language-service) that it manages using a Node.js process handler.
19+
V1.x depends on [js-graphql-language-service](https://github.com/jimkyndemeyer/js-graphql-language-service) that it manages using a Node.js process handler.
20+
21+
## Installation
22+
1. Install Node JS
23+
```
24+
# Ubuntu
25+
sudo apt install nodejs
26+
27+
# Mac OS
28+
brew install node
29+
```
30+
31+
2. Install Node JS and JS GraphQL Plugin in IntelliJ
32+
```
33+
1. File | Settings | Plugins
34+
2. Search for: nodejs and graphql
35+
3. Install both plugins and Restart
36+
```
37+
38+
3. Set Node JS Interpreter in IntelliJ
39+
```
40+
1. File | Settings | Languages & Frameworks | Node.js and NPM Node interpreter
41+
2. Add... (search for nodejs bin path in your system)
42+
43+
# Ubuntu
44+
whereis nodejs
45+
/usr/bin/nodejs
46+
47+
# Mac OS
48+
which node
49+
50+
# Windows
51+
where node
52+
```
53+
At this point JS GraphQL will recognize your `.graphql` files. However, if you wish to customize your schema, follow the FAQ.
1854

1955
## Documentation
2056

resources/META-INF/plugin.xml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,7 @@
3434
<li>2.0.0-alpha-4: Performance improvements, Relay Modern support, MarkDown documentation support, structure view, scratch files, recognize .graphqls, template placeholder support</li>
3535
<li>2.0.0-alpha-3: graphql-config support</li>
3636
<li>2.0.0-alpha-2: Language support for GraphQL draft spec as of June 2018. Compatible with Android Studio 3.1 and IntelliJ IDEA Community Edition</li>
37+
<li>1.7.3: Fixed missing syntax highlighting colors. Set default charset to UTF-8 in query result viewer. Support windows line endings in query result viewer and fixed reformat timing issue.</li>
3738
</ul>
3839
]]>
3940
</change-notes>

src/main/com/intellij/lang/jsgraphql/endpoint/ide/search/JSGraphQLEndpointDefinitionsSearchExecutor.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -28,12 +28,12 @@
2828
public class JSGraphQLEndpointDefinitionsSearchExecutor implements QueryExecutor<PsiElement, PsiElement> {
2929

3030
@Override
31-
public boolean execute(@NotNull PsiElement queryParameters, @NotNull Processor<PsiElement> consumer) {
31+
public boolean execute(@NotNull PsiElement queryParameters, @NotNull Processor<? super PsiElement> consumer) {
3232
ApplicationManager.getApplication().runReadAction((Computable) () -> JSGraphQLEndpointDefinitionsSearchExecutor.doExecute(queryParameters, consumer));
3333
return true;
3434
}
3535

36-
private static boolean doExecute(PsiElement sourceElement, final Processor<PsiElement> consumer) {
36+
private static boolean doExecute(PsiElement sourceElement, final Processor<? super PsiElement> consumer) {
3737

3838
// must be an interface definition with a named type to be applicable
3939
final Ref<JSGraphQLEndpointNamedTypeDef> sourceNamedTypeDef = new Ref<>();

0 commit comments

Comments
 (0)