Skip to content

Use GitHub admonition syntax for the warning in README#1543

Merged
dsnopek merged 1 commit into
godotengine:masterfrom
Calinou:readme-use-github-admonition-syntax
Aug 7, 2024
Merged

Use GitHub admonition syntax for the warning in README#1543
dsnopek merged 1 commit into
godotengine:masterfrom
Calinou:readme-use-github-admonition-syntax

Conversation

@Calinou

@Calinou Calinou commented Aug 5, 2024

Copy link
Copy Markdown
Member

@Calinou
Calinou requested a review from a team as a code owner August 5, 2024 17:30
@Calinou Calinou added the enhancement This is an enhancement on the current functionality label Aug 5, 2024
@Calinou
Calinou force-pushed the readme-use-github-admonition-syntax branch from b146031 to 26459dc Compare August 5, 2024 17:31
Comment thread README.md
Comment on lines +52 to +65
> [!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.

Copy link
Copy Markdown
Member Author

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.

@dsnopek dsnopek 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.

Thanks! Looks good to me :-)

@dsnopek
dsnopek merged commit 9da1e0c into godotengine:master Aug 7, 2024
@Calinou
Calinou deleted the readme-use-github-admonition-syntax branch September 2, 2024 19:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement This is an enhancement on the current functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants