-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Update tsconfig.esm.json to not target ESNext #2291
Conversation
I believe graphql#2247 was supposed to fix this, but that change does not seem to be in the main branch. Closes graphql#2240
🦋 Changeset detectedLatest commit: 59bec89 The changes in this PR will be included in the next version bump. This PR includes changesets to release 2 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
@retrodaredevil can you add a changeset so that there is a release? thank you! |
I included Now that I comparing ours, it may not need |
Thanks @retrodaredevil ! It will automatically increment the dependent modules. Looks good thanks! |
Codecov Report
@@ Coverage Diff @@
## main #2291 +/- ##
==========================================
- Coverage 65.70% 64.72% -0.99%
==========================================
Files 85 81 -4
Lines 5106 5315 +209
Branches 1631 1701 +70
==========================================
+ Hits 3355 3440 +85
- Misses 1747 1871 +124
Partials 4 4
Continue to review full report at Codecov.
|
I believe #2247 was supposed to fix this, but that change does not seem to be in the main branch.
Closes #2240