Skip to content

Commit df5848b

Browse files
saihajardatangithub-actions[bot]
authored
feat: executor from graphql-js (#4778)
* feat: executor from graphql-js * Some cleanup * Make it even more agnostic * Do not fail fast * Go * No more jsutils * Go * More * Revert "Go" This reverts commit c6a6204. * make it use tools more * fix build * remove fn in fork * changeset * bad rebase * Drop v15 * Drop v14 * Use more from executor * chore(dependencies): updated changesets for modified dependencies * Go * Yes * chore(dependencies): updated changesets for modified dependencies * Go * Relax * Go * chore(dependencies): updated changesets for modified dependencies Co-authored-by: Arda TANRIKULU <ardatanrikulu@gmail.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
1 parent e60266b commit df5848b

File tree

85 files changed

+8602
-251
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

85 files changed

+8602
-251
lines changed
Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
'@graphql-tools/delegate': patch
3+
---
4+
dependencies updates:
5+
- Added dependency [`@graphql-tools/executor@0.0.0` ↗︎](https://www.npmjs.com/package/@graphql-tools/executor/v/0.0.0) (to `dependencies`)

.changeset/brown-glasses-rule.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
'@graphql-tools/utils': minor
3+
---
4+
5+
add isIterableObject, isObjectLike, isPromise, promiseReduce, hasOwnProperty
Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
'@graphql-tools/utils': minor
3+
---
4+
5+
add `Path` util from `graphql/jsutils`
Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
'@graphql-tools/executor': patch
3+
---
4+
5+
initial release

.changeset/poor-taxis-look.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
'@graphql-tools/utils': patch
3+
---
4+
5+
improve inpsect util

.eslintrc.cjs

Lines changed: 26 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,18 @@ module.exports = {
3232
'@typescript-eslint/explicit-module-boundary-types': 'off',
3333
'default-param-last': 'off',
3434
'import/no-extraneous-dependencies': ['error', { devDependencies: ['**/*.test.ts', '**/*.spec.ts'] }],
35+
'no-restricted-imports': [
36+
'error',
37+
{
38+
paths: [
39+
{
40+
name: 'graphql',
41+
importNames: ['ExecutionResult', 'ExecutionArgs', 'execute', 'subscribe'],
42+
message: 'Please use `execute` and `subscribe` from `@graphql-tools/executro` instead.',
43+
},
44+
],
45+
},
46+
],
3547
},
3648
env: {
3749
es6: true,
@@ -68,6 +80,20 @@ module.exports = {
6880
eqeqeq: 'off',
6981
},
7082
},
83+
{
84+
files: ['packages/executor/**'],
85+
env: {
86+
jest: true,
87+
},
88+
rules: {
89+
// TODO: Enable us incrementally
90+
'@typescript-eslint/ban-ts-comment': 'off',
91+
'@typescript-eslint/no-inferrable-types': 'off',
92+
'prefer-rest-params': 'off',
93+
'no-throw-literal': 'off',
94+
'promise/param-names': 'off',
95+
},
96+
},
7197
],
7298
ignorePatterns: [
7399
'dist',

.github/workflows/benchmark.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@ jobs:
1313
name: Federation Benchmark with ${{matrix.products_size}} Products
1414
runs-on: ubuntu-latest
1515
strategy:
16+
fail-fast: false
1617
matrix:
1718
products_size: [3, 10, 50, 100, 1000]
1819
steps:

.github/workflows/tests.yml

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -34,9 +34,9 @@ jobs:
3434
name: Type Check on GraphQL v${{matrix.graphql_version}}
3535
runs-on: ubuntu-latest
3636
strategy:
37+
fail-fast: false
3738
matrix:
3839
graphql_version:
39-
- 14
4040
- 15
4141
- 16
4242
- '17.0.0-alpha.1'
@@ -64,7 +64,6 @@ jobs:
6464
os: [ubuntu-latest] # remove windows to speed up the tests
6565
node-version: [14, 16, 18]
6666
graphql_version:
67-
- 14
6867
- 15
6968
- 16
7069
- '17.0.0-alpha.1'

packages/batch-delegate/tests/basic.example.test.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
1-
import { execute, OperationTypeNode, parse } from 'graphql';
1+
import { execute } from '@graphql-tools/executor';
2+
import { OperationTypeNode, parse } from 'graphql';
23

34
import { makeExecutableSchema } from '@graphql-tools/schema';
45
import { batchDelegateToSchema } from '@graphql-tools/batch-delegate';

packages/batch-delegate/tests/withTransforms.test.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
1-
import { execute, GraphQLList, GraphQLObjectType, Kind, OperationTypeNode, parse } from 'graphql';
1+
import { execute } from '@graphql-tools/executor';
2+
import { GraphQLList, GraphQLObjectType, Kind, OperationTypeNode, parse } from 'graphql';
23

34
import { makeExecutableSchema } from '@graphql-tools/schema';
45
import { batchDelegateToSchema } from '@graphql-tools/batch-delegate';

0 commit comments

Comments
 (0)