Skip to content

Bump Gdext dep to 0.5.5, fix hot reloading - #347

Merged
nikitalita merged 4 commits into
fix-text-diffsfrom
gdext-0.5.5-hot-reload-fix
Oct 13, 2026
Merged

nikitalita merged 4 commits into
fix-text-diffsfrom
gdext-0.5.5-hot-reload-fix

Conversation

@nikitalita

Copy link
Copy Markdown
Collaborator
  • Bump gdext dependency to 0.5.5 and use api-4-7; this enables us to use the bound calls in API 4.7 instead of using call()
  • Hot reloading is working again, but was panicking due to the console subscriber thread not being torn down; keep the console subscriber as a static ref so that we can join its thread

@nikitalita
nikitalita added this pull request to stack #345 October 5, 2026 22:57

@LilithSilver LilithSilver left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks mostly good, a couple small things.

Comment thread backstitch_godot/src/tracing.rs Outdated
Comment thread backstitch_godot/src/tracing.rs Outdated
Comment thread backstitch_godot/src/tracing.rs
@nikitalita
nikitalita force-pushed the gdext-0.5.5-hot-reload-fix branch 2 times, most recently from 0a0b7b8 to db1aa8c Compare October 13, 2026 07:58
@nikitalita
nikitalita force-pushed the gdext-0.5.5-hot-reload-fix branch from db1aa8c to 5504633 Compare October 13, 2026 08:01
@nikitalita
nikitalita merged commit 5d3a820 into main Oct 13, 2026
3 of 11 checks passed
@nikitalita
nikitalita deleted the gdext-0.5.5-hot-reload-fix branch October 13, 2026 08:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants