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

build-npm/deno: use TS to find all *.ts files #3741

Merged
merged 1 commit into from
Sep 20, 2022

Conversation

IvanGoncharov
Copy link
Member

Motivation: Remove custom code for finding *.ts files and reuse code from typescript library

Motivation: Remove custom code for finding *.ts files and reuse code from typescript library
@github-actions
Copy link

Hi @IvanGoncharov, 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

@netlify
Copy link

netlify bot commented Sep 19, 2022

Deploy Preview for compassionate-pike-271cb3 ready!

Name Link
🔨 Latest commit be345af
🔍 Latest deploy log https://app.netlify.com/sites/compassionate-pike-271cb3/deploys/632887150801840008f94f44
😎 Deploy Preview https://deploy-preview-3741--compassionate-pike-271cb3.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@IvanGoncharov IvanGoncharov merged commit cbc22f2 into graphql:main Sep 20, 2022
@IvanGoncharov IvanGoncharov deleted the pr_branch branch September 20, 2022 19:38
cowboyd added a commit to thefrontside/graphgen that referenced this pull request May 4, 2023
The [test were failing when updating the website][1] because we had to
use some specifics of the way `https://esm.sh` bundled graphql-js by
reaching into the `/graphql` directory of the npm package directly.

However, now that [GraphQL-js officially supports Deno][2] we can
upgrade it. This also resolves a dependency in the gaussian
distribution library we used to pin it at a single version.

[1]: https://github.com/thefrontside/graphgen/actions/runs/4876196677/jobs/8699372244
[2]: graphql/graphql-js#3741
cowboyd added a commit to thefrontside/graphgen that referenced this pull request May 5, 2023
The [test were failing when updating the website][1] because we had to
use some specifics of the way `https://esm.sh` bundled graphql-js by
reaching into the `/graphql` directory of the npm package directly.

However, now that [GraphQL-js officially supports Deno][2] we can
upgrade it. This also resolves a dependency in the gaussian
distribution library we used to pin it at a single version.

[1]: https://github.com/thefrontside/graphgen/actions/runs/4876196677/jobs/8699372244
[2]: graphql/graphql-js#3741
cowboyd added a commit to thefrontside/graphgen that referenced this pull request May 5, 2023
The [test were failing when updating the website][1] because we had to
use some specifics of the way `https://esm.sh` bundled graphql-js by
reaching into the `/graphql` directory of the npm package directly.

However, now that [GraphQL-js officially supports Deno][2] we can
upgrade it. This also resolves a dependency in the gaussian
distribution library we used to pin it at a single version.

[1]: https://github.com/thefrontside/graphgen/actions/runs/4876196677/jobs/8699372244
[2]: graphql/graphql-js#3741
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant