Document the requirement to update the class reference when contributing
Documenting new additions has been an informal requirement for
a few months now. This makes it official.
(cherry picked from commit 6d0e8f9fb7
)
This commit is contained in:
parent
c39c3e3237
commit
c3d04167a4
|
@ -119,6 +119,22 @@ for an introduction to developing on Godot.
|
||||||
The [Contributing docs](https://docs.godotengine.org/en/latest/community/contributing/index.html)
|
The [Contributing docs](https://docs.godotengine.org/en/latest/community/contributing/index.html)
|
||||||
also have important information on the PR workflow and the code style we use.
|
also have important information on the PR workflow and the code style we use.
|
||||||
|
|
||||||
|
### Document your changes
|
||||||
|
|
||||||
|
If your pull request adds methods, properties or signals that are exposed to
|
||||||
|
scripting APIs, you **must** update the class reference to document those.
|
||||||
|
This is to ensure the documentation coverage doesn't decrease as contributions
|
||||||
|
are merged.
|
||||||
|
|
||||||
|
[Update the documentation template](https://docs.godotengine.org/en/latest/community/contributing/updating_the_class_reference.html#updating-the-documentation-template)
|
||||||
|
using your compiled binary, then fill in the descriptions.
|
||||||
|
Follow the style guide described in the
|
||||||
|
[Docs writing guidelines](https://docs.godotengine.org/en/latest/community/contributing/docs_writing_guidelines.html).
|
||||||
|
|
||||||
|
If your pull request modifies parts of the code in a non-obvious way, make sure
|
||||||
|
to add comments in the code as well. This helps other people understand the
|
||||||
|
change without having to look at `git blame`.
|
||||||
|
|
||||||
### Be nice to the Git history
|
### Be nice to the Git history
|
||||||
|
|
||||||
Try to make simple PRs that handle one specific topic. Just like for reporting
|
Try to make simple PRs that handle one specific topic. Just like for reporting
|
||||||
|
|
Loading…
Reference in New Issue