-
Notifications
You must be signed in to change notification settings - Fork 358
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
fix: Delete testing error introduced by mistake #10514
fix: Delete testing error introduced by mistake #10514
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.
Thanks @hkhalil-akamai!
I'll approve this when I get back home, I'm out running an errand 🛻 Wish I could just approve from this account 🙄 |
Coverage Report: ✅ |
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'm slightly confused about the changesets, but thanks for fixing the unintended error.
packages/manager/.changeset/pr-10514-tech-stories-1716499244555.md
Outdated
Show resolved
Hide resolved
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.
Fix looks good pending the changesets get sorted out
98cac4a
to
098c187
Compare
@mjac0bs @bnussman-akamai changesets were accidentally included from another branch -- deleted. |
Description 📝
In #10466 I accidentally committed a
Promise.reject
that was intended to test error handling inImagesDrawer
.Changes 🔄
How to test 🧪
Verify images can be edited without generating the test error.