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: Support Sprite constructor options with undefined texture #10532

Merged
merged 1 commit into from
May 14, 2024

Conversation

bigtimebuddy
Copy link
Member

Fixes #10528

Copy link

korbit-ai bot commented May 13, 2024

My review is in progress 📖 - I will have feedback for you in a few minutes!

Copy link

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit e5ccb9f:

Sandbox Source
pixi.js-sandbox Configuration

Copy link

@korbit-ai korbit-ai bot left a comment

Choose a reason for hiding this comment

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

I have reviewed your code and found 1 potential issue. To discuss my individual comments that I have added, tag me in replies using @korbit-ai.


Please react with a 👍 to my comments that you find helpful and a 👎 to those you find unhelpful - this will help me learn and improve as we collaborate.

Copy link
Member

@GoodBoyDigital GoodBoyDigital left a comment

Choose a reason for hiding this comment

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

nice! I did not know you could de-structure and have defaults! very cool 👍

@bigtimebuddy bigtimebuddy added ✅ Ready To Merge Helpful when issues are in the queue waiting to get merged. This means the PR is completed and has t and removed korbit-code-analysis labels May 13, 2024
@Zyie Zyie added the v8 label May 14, 2024
@Zyie Zyie added this pull request to the merge queue May 14, 2024
Merged via the queue into dev with commit 887526d May 14, 2024
4 checks passed
@Zyie Zyie deleted the fix/sprite-options-no-texture branch May 14, 2024 07:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✅ Ready To Merge Helpful when issues are in the queue waiting to get merged. This means the PR is completed and has t v8
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug: new Sprite gives me an error because the texture is not passed
3 participants