-
-
Notifications
You must be signed in to change notification settings - Fork 607
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
Use GitHub admonition syntax for the warning in README #1543
Use GitHub admonition syntax for the warning in README #1543
Conversation
b146031
to
26459dc
Compare
> [!WARNING] | ||
> | ||
> The GDExtension API is brand new in Godot 4.0, and is still | ||
considered in **beta** stage, despite Godot 4.0 itself being released. | ||
|
||
This applies to both the GDExtension interface header, the API JSON, and this | ||
> | ||
> This applies to both the GDExtension interface header, the API JSON, and this | ||
first-party `godot-cpp` extension. | ||
|
||
Some compatibility breakage is to be expected as GDExtension and `godot-cpp` | ||
get more used, documented, and critical issues get resolved. See the | ||
[Godot issue tracker](https://github.com/godotengine/godot/issues?q=is%3Aissue+is%3Aopen+label%3Atopic%3Agdextension) | ||
and the [godot-cpp issue tracker](https://github.com/godotengine/godot-cpp/issues) | ||
for a list of known issues, and be sure to provide feedback on issues and PRs | ||
which affect your use of this extension. | ||
> | ||
> Some compatibility breakage is to be expected as GDExtension and `godot-cpp` | ||
> get more used, documented, and critical issues get resolved. See the | ||
> [Godot issue tracker](https://github.com/godotengine/godot/issues?q=is%3Aissue+is%3Aopen+label%3Atopic%3Agdextension) | ||
> and the [godot-cpp issue tracker](https://github.com/godotengine/godot-cpp/issues) | ||
> for a list of known issues, and be sure to provide feedback on issues and PRs | ||
> which affect your use of this extension. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We might consider dropping this warning (or turning it into a simple [!NOTE]
) after Godot 4.3 is released, as compatibility breakage seems less common lately.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! Looks good to me :-)
Preview: https://github.com/Calinou/godot-cpp/blob/readme-use-github-admonition-syntax/README.md