backport(v16): Require non-empty directive locations (#4100)

graphql / graphql-js

A reference implementation of GraphQL for JavaScript

http://graphql.org/graphql-js/

MIT License

19.97k stars 2k forks source link

backport(v16): Require non-empty directive locations (#4100) #4137

Open benjie opened 4 days ago

benjie commented 4 days ago

Backport's #4100 and #4130 into 16.x.x branch.

github-actions[bot] commented 4 days ago

Hi @benjie, I'm @github-actions bot happy to help you with this PR 👋

Supported commands

Please post this commands in separate comments and only one per comment: * `@github-actions run-benchmark` - Run benchmark comparing base and merge commits for this PR * `@github-actions publish-pr-on-npm` - Build package from this PR and publish it on NPM

benjie commented 4 days ago

That's weird... CI didn't run. Closing and reopening.

benjie commented 4 days ago

All sorted. Had to fix the codecov setup.