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.

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 loader glob issues and run tests on windows #3204

Merged
merged 4 commits into from Jul 15, 2021
Merged

fix loader glob issues and run tests on windows #3204

merged 4 commits into from Jul 15, 2021

Conversation

n1ru4l
Copy link
Collaborator

@n1ru4l n1ru4l commented Jul 15, 2021 •
edited

this enables running tests on windows with node 14 and graphql 15 only.

Also resolves path issues

@changeset-bot
Copy link

changeset-bot bot commented Jul 15, 2021 •
edited

⚠️ No Changeset found

Latest commit: 1f1e867

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@n1ru4l n1ru4l changed the title run tests on windows fix loader glob issues and run tests on windows Jul 15, 2021
@n1ru4l n1ru4l merged commit e6320b0 into ardatan:master Jul 15, 2021
@n1ru4l n1ru4l deleted the run-tests-on-windows branch July 15, 2021 11:04
@theguild-bot
Copy link
Collaborator

The latest changes of this PR are available as alpha in npm (based on the declared changesets):

@graphql-tools/batch-delegate@8.0.0-alpha-e6320b00.0
@graphql-tools/batch-execute@8.0.0-alpha-e6320b00.0
@graphql-tools/delegate@8.0.0-alpha-e6320b00.0
@graphql-tools/graphql-tag-pluck@7.0.0-alpha-e6320b00.0
graphql-tools@8.0.0-alpha-e6320b00.0
@graphql-tools/jest-transform@1.0.0-alpha-e6320b00.0
@graphql-tools/links@8.0.0-alpha-e6320b00.0
@graphql-tools/load@7.0.0-alpha-e6320b00.0
@graphql-tools/apollo-engine-loader@7.0.0-alpha-e6320b00.0
@graphql-tools/code-file-loader@7.0.0-alpha-e6320b00.0
@graphql-tools/git-loader@7.0.0-alpha-e6320b00.0
@graphql-tools/github-loader@7.0.0-alpha-e6320b00.0
@graphql-tools/graphql-file-loader@7.0.0-alpha-e6320b00.0
@graphql-tools/json-file-loader@7.0.0-alpha-e6320b00.0
@graphql-tools/module-loader@7.0.0-alpha-e6320b00.0
@graphql-tools/prisma-loader@6.3.1-alpha-e6320b00.0
@graphql-tools/url-loader@7.0.0-alpha-e6320b00.0
@graphql-tools/merge@6.2.15-alpha-e6320b00.0
@graphql-tools/mock@8.1.4-alpha-e6320b00.0
@graphql-tools/node-require@6.2.5-alpha-e6320b00.0
@graphql-tools/relay-operation-optimizer@6.3.1-alpha-e6320b00.0
@graphql-tools/resolvers-composition@6.3.0-alpha-e6320b00.0
@graphql-tools/schema@8.0.0-alpha-e6320b00.0
@graphql-tools/stitch@8.0.0-alpha-e6320b00.0
@graphql-tools/stitching-directives@2.0.0-alpha-e6320b00.0
@graphql-tools/utils@8.0.0-alpha-e6320b00.0
@graphql-tools/webpack-loader@6.5.0-alpha-e6320b00.0
@graphql-tools/wrap@8.0.0-alpha-e6320b00.0

ardatan pushed a commit that referenced this pull request Jul 25, 2021
@n1ru4l @ardatan
fix loader glob issues and run tests on windows (#3204)
cd5d115 
* run tests on windows

* fix windows compat

* only run windows for latest graphql version and node LTS

* refactor logic into function
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@n1ru4l @theguild-bot