-
Notifications
You must be signed in to change notification settings - Fork 201
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
Introduce extended schema plugin. #792
Conversation
Codecov Report
@@ Coverage Diff @@
## 8.x-4.x #792 +/- ##
===========================================
- Coverage 79.89% 79.3% -0.6%
- Complexity 535 540 +5
===========================================
Files 74 75 +1
Lines 1194 1203 +9
===========================================
Hits 954 954
- Misses 240 249 +9
Continue to review full report at Codecov.
|
use GraphQL\Language\Parser; | ||
use GraphQL\Utils\SchemaExtender; | ||
|
||
abstract class SdlExtendedSchemaPluginBase extends SdlSchemaPluginBase { |
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.
we should have a class comment why we need this and what it does.
* The parsed extended schema document. | ||
*/ | ||
protected function getExtendedSchemaDocument() { | ||
// Only use caching of the parsed document if aren't in development mode. |
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.
"if we aren't in development mode"
Sweet, thanks |
No description provided.