Skip to content

docs: Update README.md #529

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 4 commits into from
Jan 27, 2025
Merged

docs: Update README.md #529

merged 4 commits into from
Jan 27, 2025

Conversation

KANAjetzt
Copy link
Member

We can remove the class_name restriction.

@KANAjetzt KANAjetzt added the documentation Improvements or additions to documentation label Jan 26, 2025
@KANAjetzt KANAjetzt added this to the 3.x - v6.3.0 milestone Jan 26, 2025
@KANAjetzt KANAjetzt requested a review from Qubus0 January 26, 2025 22:03
@KANAjetzt KANAjetzt self-assigned this Jan 26, 2025
@KANAjetzt KANAjetzt added this pull request to the merge queue Jan 27, 2025
Merged via the queue into development with commit ccadca5 Jan 27, 2025
@KANAjetzt KANAjetzt mentioned this pull request Jan 27, 2025
Merged
KANAjetzt added a commit that referenced this pull request Jan 27, 2025
@KANAjetzt
v6.3.0 (#528)
92cd37b 
# Minor Release v6.3.0

Minor update to fix issues with script extension sorting, as well as
potential issues with translations, configurations, and user profiles.

## Key PRs
- #357
- #358
- #520
- #352

## Change Log

### New Features

- #352
- #358

### Fixes
- #357
- #520 
- #522 
- #523
- #524

### Refactors
- #521
- #526

### Tests
- #360 

### Miscellaneous
- #525
- #527
- #529
@KANAjetzt KANAjetzt deleted the KANAjetzt-patch-1 branch January 27, 2025 11:43
@Exerionius
Copy link

  1. If mentions of class_name limitations were removed, does it mean that the issue was resolved?
  2. If the issue isn't resolved but there's a workaround, is it documented somewhere? Besides "just don't use class names".
  3. How it relates to this? take_over_path and extending the same named class twice godotengine/godot#83542

@Qubus0
Copy link
Collaborator

Qubus0 commented Jan 27, 2025

Hey @Exerionius, the class_name issue persists, but we've worked around it with script hooks, which is essentially runtime modification and patching of scripts. Doc update still work in progress, but you can see some drafts here https://qubus0.github.io/gmlwiki/guides/modding/script_hooks/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Qubus0 Qubus0 Qubus0 approved these changes

Assignees

@KANAjetzt KANAjetzt

Labels
documentation Improvements or additions to documentation
Projects
None yet
Milestone
3.x - v6.3.0
Development

Successfully merging this pull request may close these issues.
3 participants
@KANAjetzt @Exerionius @Qubus0