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

Removing deprecated editor_plugin attribute key #86

Merged
merged 1 commit into from
Mar 25, 2025

Conversation

reshen
Copy link
Contributor

@reshen reshen commented Mar 25, 2025

In godot-rust/gdext#884, editor_plugin was
deprecated (automatically implied by EditorPlugin), and raises a warning
when provided. Updating the docs to reflect that change.

In godot-rust/gdext#884, editor_plugin was
deprecated (automatically implied by EditorPlugin), and raises a warning
when provided. Updating the docs to reflect that change.
@Bromeon Bromeon added the outdated-api Information that is no longer accurate label Mar 25, 2025
Copy link
Member

@Bromeon Bromeon left a comment

Choose a reason for hiding this comment

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

Thanks!

<!-- TODO: more plugins from https://docs.godotengine.org/en/stable/tutorials/plugins/editor/index.html -->
Copy link
Member

Choose a reason for hiding this comment

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

What changed here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Looks like vim fixed something automatically for me here -- the file was missing a trailing new line which was added.

I'm surprised that it's not explicitly rendered here in github's diff, but git cli diffs tell a clearer story of what changed:

❯ git diff upstream/master src/recipes/editor-plugin/index.md

...

<!-- TODO: more plugins from https://docs.godotengine.org/en/stable/tutorials/plugins/editor/index.html -->
\ No newline at end of file
<!-- TODO: more plugins from https://docs.godotengine.org/en/stable/tutorials/plugins/editor/index.html -->

@Bromeon Bromeon merged commit 5f1570c into godot-rust:master Mar 25, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
outdated-api Information that is no longer accurate
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants