You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Dec 19, 2019. It is now read-only.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
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.
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
@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.
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?
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.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description (*)
These declarations remove a lot of warnings from the PHPStorm GraphQl plugin while editing schema.graphqls files.
References:
Contribution checklist (*)