Skip to content

Comments

develop#237

Merged
ienaga merged 7 commits intomainfrom
develop
Nov 25, 2025
Merged

develop#237
ienaga merged 7 commits intomainfrom
develop

Conversation

@ienaga
Copy link
Member

@ienaga ienaga commented Nov 25, 2025

No description provided.

Copilot AI review requested due to automatic review settings November 25, 2025 13:31
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the Next2D Player to version 2.12.0 with a bug fix for Stage's addChild method and adds comprehensive test coverage for the Stage class.

  • Version bumped from 2.11.0 to 2.12.0 across package files
  • Fixed root map logic in Stage.addChild() to correctly set the display object as its own root
  • Added comprehensive unit tests for the Stage class
  • Updated dev dependencies (@typescript-eslint packages and vitest)

Reviewed changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
src/index.ts Updated version string in console log to 2.12.0
packages/display/src/Stage.ts Fixed addChild method to set root map after parent addChild call, ensuring correct root assignment
packages/display/src/Stage.test.ts Added comprehensive test suite for Stage class covering initialization, properties, and addChild behavior
package.json Updated package version to 2.12.0 and dev dependencies
package-lock.json Updated lockfile with new version and dependency resolutions

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

ienaga and others added 2 commits November 25, 2025 22:48
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@ienaga ienaga merged commit c102b65 into main Nov 25, 2025
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant