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

Inline @josephg/resolvable dependency #7900

Merged
merged 4 commits into from
Jul 22, 2024
Merged

Conversation

trevor-scheer
Copy link
Member

Issue #7895 reports that AS4 fails to build for ESM due to the resolvable dependency.

I'm actually unsure why this is happening though, since the shipped JS appears to be ESM compatible:
https://unpkg.com/@josephg/resolvable@1.0.1/index.js

@macrozone would you mind testing the builds from this PR (see Codesandbox CI comment) and letting me know if this resolves the issue in your project?

Copy link

netlify bot commented Jul 11, 2024

Deploy Preview for apollo-server-docs ready!

Name Link
🔨 Latest commit 4602e65
🔍 Latest deploy log https://app.netlify.com/sites/apollo-server-docs/deploys/66900edba46ca300086e341b
😎 Deploy Preview https://deploy-preview-7900--apollo-server-docs.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 configuration.

Copy link

codesandbox-ci bot commented Jul 11, 2024

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

@macrozone
Copy link

@trevor-scheer I can confirm that this resolves the issue!

@davidhasek1
Copy link

@trevor-scheer Thank you for this fix.

@trevor-scheer trevor-scheer merged commit 86d7111 into main Jul 22, 2024
22 checks passed
@trevor-scheer trevor-scheer deleted the trevor/inline-resolvable branch July 22, 2024 20:48
@github-actions github-actions bot mentioned this pull request Jul 22, 2024
trevor-scheer pushed a commit that referenced this pull request Jul 22, 2024
This PR was opened by the [Changesets
release](https://github.com/changesets/action) GitHub action. When
you're ready to do a release, you can merge this and the packages will
be published to npm automatically. If you're not ready to do a release
yet, that's fine, whenever you add more changesets to main, this PR will
be updated.


# Releases
## @apollo/server-integration-testsuite@4.10.5

### Patch Changes

- [#7821](#7821)
[`b2e15e7`](b2e15e7)
Thanks [@renovate](https://github.com/apps/renovate)! - Non-major
dependency updates

- [#7900](#7900)
[`86d7111`](86d7111)
Thanks [@trevor-scheer](https://github.com/trevor-scheer)! - Inline a
small dependency that was causing build issues for ESM projects

- Updated dependencies
\[[`b2e15e7`](b2e15e7),
[`86d7111`](86d7111)]:
    -   @apollo/server@4.10.5

## @apollo/server@4.10.5

### Patch Changes

- [#7821](#7821)
[`b2e15e7`](b2e15e7)
Thanks [@renovate](https://github.com/apps/renovate)! - Non-major
dependency updates

- [#7900](#7900)
[`86d7111`](86d7111)
Thanks [@trevor-scheer](https://github.com/trevor-scheer)! - Inline a
small dependency that was causing build issues for ESM projects

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 23, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants