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

fix: Delete testing error introduced by mistake #10514

Merged
merged 1 commit into from
May 24, 2024

Conversation

hkhalil-akamai
Copy link
Contributor

Description 📝

In #10466 I accidentally committed a Promise.reject that was intended to test error handling in ImagesDrawer.

Changes 🔄

  • Delete testing error

How to test 🧪

Verify images can be edited without generating the test error.

@hkhalil-akamai hkhalil-akamai added the Bug Fixes for regressions or bugs label May 23, 2024
@hkhalil-akamai hkhalil-akamai self-assigned this May 23, 2024
@hkhalil-akamai hkhalil-akamai requested a review from a team as a code owner May 23, 2024 20:58
@hkhalil-akamai hkhalil-akamai requested review from hana-akamai, AzureLatte and jdamore-linode and removed request for a team May 23, 2024 20:58
Copy link
Contributor

@jdamore-linode jdamore-linode left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @hkhalil-akamai!

@bnussman
Copy link
Member

I'll approve this when I get back home, I'm out running an errand 🛻

Wish I could just approve from this account 🙄

Copy link

github-actions bot commented May 23, 2024

Coverage Report:
Base Coverage: 81.74%
Current Coverage: 81.74%

Copy link
Contributor

@mjac0bs mjac0bs left a 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.

Copy link
Member

@bnussman-akamai bnussman-akamai left a 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

@AzureLatte AzureLatte removed their request for review May 24, 2024 13:15
@hkhalil-akamai
Copy link
Contributor Author

@mjac0bs @bnussman-akamai changesets were accidentally included from another branch -- deleted.

@hkhalil-akamai hkhalil-akamai merged commit 5bf015e into linode:develop May 24, 2024
34 checks passed
@hkhalil-akamai hkhalil-akamai deleted the images-drawer-error branch May 24, 2024 15:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Fixes for regressions or bugs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants