-
Notifications
You must be signed in to change notification settings - Fork 27
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
chore(deps): update dependency graphql-cli to v4 #1552
Conversation
🦋 Changeset is good to goLatest commit: ecd9043 We got this. This PR includes changesets to release 7 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/commercetools/merchant-center-application-kit/dl1mblxv1 |
0c58cac
to
599efde
Compare
599efde
to
8581050
Compare
8581050
to
bcddc47
Compare
bcddc47
to
23e9158
Compare
23e9158
to
c3c79c1
Compare
c3c79c1
to
42ca816
Compare
42ca816
to
e040b8a
Compare
e040b8a
to
e53a0b6
Compare
e53a0b6
to
cb0be76
Compare
97b600b
to
d846fff
Compare
d846fff
to
077f6b8
Compare
077f6b8
to
090b62a
Compare
29d0d1e
to
3403d27
Compare
FYI: migration steps https://github.com/Urigo/graphql-cli/blob/master/docs/MIGRATION.md |
eb3f559
to
8658857
Compare
@@ -0,0 +1,144 @@ | |||
module.exports = { |
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.
This is now a JS file because we need to use require()
to load the graphql schema files
{ | ||
files: ['**/*.mc.graphql'], | ||
rules: { | ||
'graphql/template-strings': [ | ||
'error', | ||
{ | ||
env: 'literal', | ||
schemaJson: require('./schemas/mc.json'), | ||
}, | ||
], | ||
}, | ||
}, | ||
{ | ||
files: ['**/*.ctp.graphql'], | ||
rules: { | ||
'graphql/template-strings': [ | ||
'error', | ||
{ | ||
env: 'literal', | ||
schemaJson: require('./schemas/ctp.json'), | ||
}, | ||
], | ||
}, | ||
}, | ||
{ | ||
files: ['**/*.settings.graphql'], | ||
rules: { | ||
'graphql/template-strings': [ | ||
'error', | ||
{ | ||
env: 'literal', | ||
schemaJson: require('./schemas/settings.json'), | ||
}, | ||
], | ||
}, | ||
}, | ||
{ | ||
files: ['**/*.proxy.graphql'], | ||
rules: { | ||
'graphql/template-strings': [ | ||
'error', | ||
{ | ||
env: 'literal', | ||
schemaJson: require('./schemas/proxy.json'), | ||
}, | ||
], | ||
}, | ||
}, |
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.
This is the new part. See apollographql/eslint-plugin-graphql#246
@@ -0,0 +1,163 @@ | |||
projects: |
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.
The new config does both schema introspection and types generation.
'*.mc.graphql': () => 'yarn generate-types:mc', | ||
'*.ctp.graphql': () => 'yarn generate-types:ctp', | ||
'*.settings.graphql': () => 'yarn generate-types:settings', | ||
'*.proxy.graphql': () => 'yarn generate-types:proxy', |
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.
For each graphql "target", we run its own generate script.
About the () =>
: https://github.com/okonet/lint-staged#example-run-tsc-on-changes-to-typescript-files-but-do-not-pass-any-filename-arguments
This PR contains the following updates:
3.0.14
->4.0.0
Release Notes
Urigo/graphql-cli
v4.0.0
Compare Source
@graphql-cli
scoped packagesvalidate
,coverage
,diff
andsimilar
commands - GraphQL Inspectorcodegen
commandplayground
withserve
commandget-schema
withintrospect
commandRenovate configuration
📅 Schedule: "before 3am on Monday" (UTC).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻️ Rebasing: Renovate will not automatically rebase this PR, because other commits have been found.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR has been generated by WhiteSource Renovate. View repository job log here.