-
Notifications
You must be signed in to change notification settings - Fork 10.3k
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
docs(gatsby-plugin-image): Initial migration guide #29036
Conversation
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.
This is definitely on the right track! 🙌 I think all the right pieces of information are here, but there might be a better way to group them. The way I'm chunking this content in my head looks something like:
- Introduction (there's a new plugin!)
- What changed?
- New Syntax
- API changes
- Unsupported usage
- How to migrate?
- Install
- Codemod instructions
- Codemod assumptions
- Recommended changes (I got the sense that these were additional changes needed on top of the codemod?)
What do you think?
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.
Just a few comments. I've done them as change suggestions, but that's just as it's easier. I'm not expecting you to use my words
Co-authored-by: Matt Kane <[email protected]>
Co-authored-by: Matt Kane <[email protected]>
Co-authored-by: Matt Kane <[email protected]>
Co-authored-by: Matt Kane <[email protected]>
Co-authored-by: Matt Kane <[email protected]>
Co-authored-by: Matt Kane <[email protected]>
Co-authored-by: Matt Kane <[email protected]>
Co-authored-by: Matt Kane <[email protected]>
Co-authored-by: Matt Kane <[email protected]>
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.
@meganesu should check too, but this looks great to me.
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.
This looks awesome! Left a bunch of copy edits, but after those this looks ready to me! Great job 🎉 ✨
Co-authored-by: Megan Sullivan <[email protected]>
Co-authored-by: Megan Sullivan <[email protected]>
Co-authored-by: Megan Sullivan <[email protected]>
Co-authored-by: Megan Sullivan <[email protected]>
Co-authored-by: Megan Sullivan <[email protected]>
Co-authored-by: Megan Sullivan <[email protected]>
Co-authored-by: Megan Sullivan <[email protected]>
Co-authored-by: Megan Sullivan <[email protected]>
Co-authored-by: Megan Sullivan <[email protected]>
Co-authored-by: Megan Sullivan <[email protected]>
Co-authored-by: Megan Sullivan <[email protected]>
Co-authored-by: Megan Sullivan <[email protected]>
Co-authored-by: Megan Sullivan <[email protected]>
Co-authored-by: Megan Sullivan <[email protected]>
Co-authored-by: Megan Sullivan <[email protected]>
Co-authored-by: Megan Sullivan <[email protected]>
Co-authored-by: Megan Sullivan <[email protected]>
Co-authored-by: Megan Sullivan <[email protected]>
Co-authored-by: Matt Kane <[email protected]>
Co-authored-by: Matt Kane <[email protected]>
Co-authored-by: Matt Kane <[email protected]>
Co-authored-by: Matt Kane <[email protected]>
Co-authored-by: Matt Kane <[email protected]>
Co-authored-by: Matt Kane <[email protected]>
This is a draft of the migration guide.
[ch22957]