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
Document that non-spec changes should get an 'internal' changelog entry #1342
Conversation
``changelogs/``. For instance, news fragments for the client-server API are stored | ||
under ``changelogs/client_server/newsfragments``. Any changes to the repository that do | ||
not affect the spec content itself, such as changes to the build script, formatting, CSS, | ||
etc. should be documented under ``changelogs/internal/newsfragments``. | ||
|
||
To create a changelog entry, create a file named in the format ``prNumber.type`` in | ||
the ``newsfragments`` directory. The ``type`` can be one of the following: |
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.
Do we want to provide any additional guidance on what types are generally used for internal changes? So far I only see "clarification" and "feature".
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.
Hopefully it should be relatively self-explanatory, although I'm surprised there aren't more bugfixes.
``changelogs/``. For instance, news fragments for the client-server API are stored | ||
under ``changelogs/client_server/newsfragments``. Any changes to the repository that do | ||
not affect the spec content itself, such as changes to the build script, formatting, CSS, | ||
etc. should be documented under ``changelogs/internal/newsfragments``. | ||
|
||
To create a changelog entry, create a file named in the format ``prNumber.type`` in | ||
the ``newsfragments`` directory. The ``type`` can be one of the following: |
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.
Hopefully it should be relatively self-explanatory, although I'm surprised there aren't more bugfixes.
Updates the contributing documentation to mention that we now support "internal" changelog entries after #1194.
Preview: https://pr1342--matrix-spec-previews.netlify.app