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.

Sign up for GitHub

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

Jump to bottom

fix: eslint warining fixes #1360

Merged

acao merged 1 commit into graphql:master from Gumichocopengin8:feature/bug-fixes

Feb 22, 2020

Contributor

Gumichocopengin8 commented Feb 22, 2020

/Users/keita/Desktop/starPlatinum/graphQL/graphiql/packages/graphql-language-service-server/src/__tests__/GraphQLCache-test.ts
  34:11  warning  'configDir' is already declared in the upper scope  no-shadow

/Users/keita/Desktop/starPlatinum/graphQL/graphiql/resources/test.config.js
  1:11  warning  'jsdom' is defined but never used  no-unused-vars

✖ 2 problems (0 errors, 2 warnings)

fixed this warning


          fix: eslint warining fixes

2895aa5

Gumichocopengin8 requested a review from acao

February 22, 2020 18:23

Member

acao commented Feb 22, 2020

thanks @Gumichocopengin8 !

acao merged commit 84d4821 into graphql:master

Gumichocopengin8 deleted the feature/bug-fixes branch

February 22, 2020 20:35

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment