-
Notifications
You must be signed in to change notification settings - Fork 2.7k
add a removals codemod
#12838
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
add a removals codemod
#12838
Conversation
🦋 Changeset detectedLatest commit: 898d51f The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
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 |
commit: |
size-limit report 📦
|
✅ Docs preview readyThe preview is ready to be viewed. View the preview File Changes 26 new, 66 changed, 5 removedBuild ID: 731b5e09c43e02ff1a5f0784 URL: https://www.apollographql.com/docs/deploy-preview/731b5e09c43e02ff1a5f0784 |
f9d37d0 to
6265da5
Compare
69f4806 to
56e2502
Compare
4c80e90 to
5944a3e
Compare
jerelmiller
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's connect and bike shed on the entrypoint name, but I love this PR!
package.json
Outdated
| "default": "./src/utilities/environment/index.ts" | ||
| } | ||
| }, | ||
| "./zz_removals": "./src/zz_removals.ts" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I love every part of this PR except this name 😆. What about something like v4-migration?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah I hate it too, I just wanted it to be alphabetically sorted to the end so users don't accidentally use it.
If we don't plan for entrypoints starting with w, x, y, or z, v4-migration sounds great!
src/zz_removals.ts
Outdated
| * docblocks will explain the removal and suggest alternatives. | ||
| */ | ||
|
|
||
| export declare namespace Removals { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note for the upgrade guide: we will probably want to mention that any non-type exported from this entrypoint will cause runtime errors.
642973d to
898d51f
Compare
No description provided.