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 scene root entity destroy bug #1062

Merged
merged 2 commits into from
Sep 19, 2022
Merged

Fix scene root entity destroy bug #1062

merged 2 commits into from
Sep 19, 2022

Conversation

GuoLei1990
Copy link
Member

Please check if the PR fulfills these requirements

  • The commit message follows our guidelines
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

What is the current behavior? (You can also link to an open issue here)

What is the new behavior (if this is a feature change)?

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)

Other information:

@GuoLei1990 GuoLei1990 added the bug Something isn't working label Sep 19, 2022
@GuoLei1990 GuoLei1990 self-assigned this Sep 19, 2022
@GuoLei1990
Copy link
Member Author

@codecov-commenter
Copy link

Codecov Report

Base: 33.70% // Head: 33.69% // Decreases project coverage by -0.00% ⚠️

Coverage data is based on head (dd92c52) compared to base (5982faf).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1062      +/-   ##
==========================================
- Coverage   33.70%   33.69%   -0.01%     
==========================================
  Files         334      334              
  Lines       14458    14463       +5     
  Branches     2216     2217       +1     
==========================================
+ Hits         4873     4874       +1     
- Misses       9058     9060       +2     
- Partials      527      529       +2     
Impacted Files Coverage Δ
packages/core/src/Entity.ts 75.74% <100.00%> (+0.31%) ⬆️
packages/math/src/Color.ts 89.28% <0.00%> (-3.58%) ⬇️
packages/core/src/physics/CharacterController.ts 2.63% <0.00%> (-0.15%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@GuoLei1990 GuoLei1990 merged commit c3b2d6f into galacean:main Sep 19, 2022
GuoLei1990 added a commit to GuoLei1990/galacean-engine that referenced this pull request Sep 22, 2022
* main:
  v0.8.0-beta.26
  feat: shader add properties get ability (galacean#1060)
  Fix scene root entity destroy bug (galacean#1062)
  Fix call `engine.destroy()` crash  in `Script` (galacean#1059)
  Fix physics system stablity (galacean#1047)
  v0.8.0-beta.25
  Fix script onDisable() error call bug (galacean#1055)
  fix: sprite transform error when rotates (galacean#1053)
  feat: remove throw (galacean#1052)
  Update README.md
  Update README.md
  v0.8.0-beta.24
  Fix `TextRenderer` render error (galacean#1040)
  v0.8.0-beta.23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants