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

Validate that virtual extension classes require #[class(tool)] #850

Merged
merged 1 commit into from
Aug 11, 2024

Conversation

Bromeon
Copy link
Member

@Bromeon Bromeon commented Aug 10, 2024

Adds validation for an issue brought up in #826 (comment).

@TitanNano you might want to have a look?

@Bromeon Bromeon added quality-of-life No new functionality, but improves ergonomics/internals c: register Register classes, functions and other symbols to GDScript labels Aug 10, 2024
@Bromeon Bromeon force-pushed the qol/require-tool-for-virtual-extensions branch from 011d157 to 62bbf1d Compare August 10, 2024 22:27
@GodotRust
Copy link

API docs are being generated and will be shortly available at: https://godot-rust.github.io/docs/gdext/pr-850

@TitanNano
Copy link
Contributor

Looks good. Tried it with ScriptExtension and ScriptLanguageExtension.

@Bromeon Bromeon added this pull request to the merge queue Aug 11, 2024
Merged via the queue into master with commit 05446f6 Aug 11, 2024
14 checks passed
@Bromeon Bromeon deleted the qol/require-tool-for-virtual-extensions branch August 11, 2024 07:12
@TitanNano
Copy link
Contributor

@Bromeon I see I'm just a bit too late. It just came to my mind that we perhaps also want to include EditorPlugin here. There are also some import Plugin classes that are only used in the editor.

@Bromeon
Copy link
Member Author

Bromeon commented Aug 11, 2024

Hm good idea -- would you mind opening a PR for it? Since you already have some classes in mind 🤔
I think the structure could follow the changes from this PR quite closely.

@TitanNano
Copy link
Contributor

Yeah I can do that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c: register Register classes, functions and other symbols to GDScript quality-of-life No new functionality, but improves ergonomics/internals
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants