TypeError: Cannot convert undefined or null to object

With the new 1.0.0 (and Node.js v12.18.3) I’m having an error:

It looks like you may have hit a bug in graphql-schema-linter.

It would be super helpful if you could report this here: https://github.com/cjoudrey/graphql-schema-linter/issues/new

TypeError: Cannot convert undefined or null to object
    at Function.entries (<anonymous>)
    at ignoreListErrorFilter (/Users/me/Project/node_modules/graphql-schema-linter/lib/validator.js:143:39)
    at validateSchemaDefinition (/Users/me/Project/node_modules/graphql-schema-linter/lib/validator.js:72:68)
    at run (/Users/me/Project/node_modules/graphql-schema-linter/lib/runner.js:68:58)

This is how I’m running it:

#!/usr/bin/env bash
set -e

# Enable `**` globstar
shopt -s globstar

# Also append AWS AppSync definitions
# See https://github.com/cjoudrey/graphql-schema-linter/issues/165
graphql-schema-linter \
  --comment-descriptions \
   schema/**/*.graphql graphql-schema-linter-aws.graphql

Note that this is working fine with 0.5.0

[connorshea]

It’s fixed for me if I add "ignore": {} to my linter config, so if I had to guess I think it’s this line that’s doing it: https://github.com/cjoudrey/graphql-schema-linter/commit/5dc06315cfdb4cf2b42df2b628df80bbf6c1b93c#diff-91ba5f4df12e1b6b0629662164b3f14eR32

Just need to add a default value of {} for ignore

Should probably run the tests using a config file that doesn’t have the ignore field as well 😃

[cjoudrey]

Thanks all for your patience. This should now be fixed in v1.0.1.

Also thanks to @paramjitkaur for opening a pull request with a fix. I appreciate the help! 😄