Skip to content
This repository has been archived by the owner on Dec 19, 2019. It is now read-only.

Declare @doc and @resolver directives #775

Merged
merged 4 commits into from
Jul 30, 2019

Conversation

Vinai
Copy link
Contributor

@Vinai Vinai commented Jul 16, 2019

Description (*)

These declarations remove a lot of warnings from the PHPStorm GraphQl plugin while editing schema.graphqls files.

References:

Contribution checklist (*)

  • Pull request has a meaningful description of its purpose
  • All commits are accompanied by meaningful commit messages
  • All new or changed code is covered with unit/integration tests (if applicable)
  • All automated tests passed successfully (all builds are green)

These declarations remove a lot of warnings from the PHPStorm
GraphQl plugin while editing schema.graphqls files.
@Vinai
Copy link
Contributor Author

Vinai commented Jul 16, 2019

Example screenshots without the declarations:

00-without-declarations

Example screenshots with the declarations:

01-with-declarations

Vinai added 2 commits July 17, 2019 11:07
The class `\GraphQL\Language\Parser` from the webonyx/graphql-php package throws an `Syntax Error: Unexpected Name "INTERFACE"` exception when the token `INTERFACE` comes last in the `DirectiveLocations` part of a directive declaration.
Adding `OBJECT` as the last item removes the error.
@magento-engcom-team
Copy link
Contributor

Hi @lenaorobei, thank you for the review.
ENGCOM-5460 has been created to process this Pull Request
✳️ @lenaorobei, could you please add one of the following labels to the Pull Request?

Label Description
Auto-Tests: Covered All changes in Pull Request is covered by auto-tests
Auto-Tests: Not Covered Changes in Pull Request requires coverage by auto-tests
Auto-Tests: Not Required Changes in Pull Request does not require coverage by auto-tests

The `@cache` directive is available since 2.3.2.
@Vinai
Copy link
Contributor Author

Vinai commented Jul 18, 2019

@lenaorobei I updated the PR despite your existing approval to add a declaration for the @cache directive, too. Please have another look if you still think it's okay - thank you.

@lenaorobei
Copy link
Contributor

@magento run all tests

@Vinai
Copy link
Contributor Author

Vinai commented Jul 23, 2019

Seems like the builds is still stalled. Do you have an ETA when they will work again?
Will the stalled builds be restarted automatically, or should I take some action?

@lenaorobei
Copy link
Contributor

@Vinai we are experiencing some issues with builds and working on the fix. I'll manually re-request builds run. Thanks for understanding and patience.

@Vinai
Copy link
Contributor Author

Vinai commented Jul 23, 2019

Thank you @lenaorobei

@lenaorobei
Copy link
Contributor

@magento run all tests

@lenaorobei
Copy link
Contributor

@magento run Functional Tests build, WebAPI Tests build

@Vinai
Copy link
Contributor Author

Vinai commented Jul 24, 2019

@lenaorobei The WebAPI Tests build seems to be (still?) hanging, it's been 17 hours.
Could you comment with the magic command again please?

@lenaorobei
Copy link
Contributor

@magento run WebAPI Tests build

@magento-engcom-team magento-engcom-team merged commit 24da88e into magento:2.3-develop Jul 30, 2019
@ghost
Copy link

ghost commented Jul 30, 2019

Hi @Vinai, thank you for your contribution!
Please, complete Contribution Survey, it will take less than a minute.
Your feedback will help us to improve contribution process.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants