Skip to content
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

expectJSON: improve readability #3400

Merged
merged 1 commit into from
Dec 3, 2021
Merged

Conversation

IvanGoncharov
Copy link
Member

No description provided.

@IvanGoncharov IvanGoncharov added the PR: polish 💅 PR doesn't change public API or any observed behaviour label Dec 3, 2021
@IvanGoncharov IvanGoncharov merged commit cbbff7f into graphql:main Dec 3, 2021
@IvanGoncharov IvanGoncharov deleted the pr_branch branch December 3, 2021 18:01
yaacovCR added a commit to yaacovCR/graphql-js that referenced this pull request Dec 6, 2021
* execute: fix spec section names in comments (graphql#3376)

* lexer-tests: Use tildas as invalid characters (graphql#3377)

* Preserve non-error values thrown from resolvers (graphql#3384)

* Add support for Node17 (graphql#3386)

* Update package-lock.json (graphql#3387)

* Drop "eslint-plugin-istanbul" and implement as internal ESLint rule (graphql#3388)

* package.json: Drop unused '@babel/eslint-parser' (graphql#3389)

* Update deps (graphql#3390)

* Enable '@typescript-eslint/switch-exhaustiveness-check' rule (graphql#3391)

* Remove $FlowFixMe comments (graphql#3392)

* typeFromAST: use exhaustive switch and remove invariant (graphql#3396)

* Update doc examples to reflect the current API (graphql#3393)

* Switch coverage from nyc to c8 (graphql#3398)

* Update deps (graphql#3399)

* expectJSON: improve readability (graphql#3400)

* tests: Improve formating of strings with 'dedent' tag (graphql#3401)

* ci: add check that 'package-lock.json' doesn't have conflicts (graphql#3403)

* github/workflows: simplify npm cache setup (graphql#3404)

* Use for '--ignore-scripts' for all `npm ci` & `npm install` (graphql#3405)

* Simplify code by replacing Object.entries with Object.keys (graphql#3406)

* ci: add check for unnessary duplicates in package-lock.json (graphql#3407)

* update lockfile

* Fix ci merge

* fix package-lock file

...starting again from most current graphql-js

Co-authored-by: Alex Reilly <fabiobean2@gmail.com>
Co-authored-by: Ivan Goncharov <ivan.goncharov.ua@gmail.com>
Co-authored-by: Paul Serraino <pserraino99@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: polish 💅 PR doesn't change public API or any observed behaviour
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants