Skip to content

feat(core-editor): hot reload entity component from save#253

Open
Tchips46 wants to merge 15 commits intomainfrom
feat/core-editor
Open

feat(core-editor): hot reload entity component from save#253
Tchips46 wants to merge 15 commits intomainfrom
feat/core-editor

Conversation

@Tchips46
Copy link
Copy Markdown
Contributor

What does this PR do?

Resolves #231

How do you test this PR?

@Tchips46 Tchips46 self-assigned this Mar 25, 2026
@Tchips46 Tchips46 requested a review from Exeloo as a code owner March 25, 2026 07:02
@Tchips46 Tchips46 requested a review from MartinFillon as a code owner March 25, 2026 07:09
@github-actions github-actions bot added the packages:ecs-lib Related to Abstract ECS library label Mar 25, 2026
@Tchips46 Tchips46 changed the title Feat/core editor feat(core-editor): hot reload entity component from save Mar 25, 2026
@github-actions github-actions bot added packages:ecs-client Related to ECSClient library packages:ecs-server Related to ECSServer library labels Mar 26, 2026
Copy link
Copy Markdown
Member

@Exeloo Exeloo left a comment

Choose a reason for hiding this comment

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

You must add core editor to contribution tools (labels, labeler, issue templates)

@Exeloo
Copy link
Copy Markdown
Member

Exeloo commented Mar 26, 2026

Also wouldn't be better to import and extends from base core features that doesn't change ? Like for class or functions that doesn't change use the core one and for class which changed, extends the core one to use old method that doesn't change

MartinFillon
MartinFillon previously approved these changes Mar 27, 2026
Copy link
Copy Markdown
Contributor

@MartinFillon MartinFillon left a comment

Choose a reason for hiding this comment

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

lgtm

@github-actions github-actions bot added the packages:core Related to Core library label Mar 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

packages:core Related to Core library packages:ecs-client Related to ECSClient library packages:ecs-lib Related to Abstract ECS library packages:ecs-server Related to ECSServer library

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Create a core for editor

3 participants