mirror of
https://github.com/ChristianLight/tutor.git
synced 2024-12-12 06:07:56 +00:00
docs: more precise contribution instructions
This commit is contained in:
parent
a9f0d021b1
commit
fdd230f353
@ -105,13 +105,24 @@ indicates that this filter definition can be removed as soon as Tutor v26.0.0.
|
||||
Contributing to Tutor
|
||||
---------------------
|
||||
|
||||
Third-party contributions to Tutor and its plugins are more than welcome! Just make sure to follow these guidelines:
|
||||
Contributions to Tutor and its plugins are highly encouraged. Please adhere to the following guidelines:
|
||||
|
||||
- Outside of obvious bugs, contributions should be discussed first in the `official Open edX forum <https://discuss.openedx.org>`__.
|
||||
- Once we agree on a high-level solution, you should open a pull request on the `Tutor repository <https://github.com/overhangio/tutor/pulls>`__ or the corresponding plugin.
|
||||
- Make sure that all tests pass by running ``make test`` (see above).
|
||||
- If your PR is in the Tutor core repository, add a changelog entry by running ``make changelog-entry``. Edit the new file and follow the formatting instructions that it contains.
|
||||
- Write a good Git commit title and message: explain why you are making this change, what problem you are solving and which solution you adopted. Link to the relevant conversation topics in the forums and describe your use case. We *love* long, verbose descriptions :) As for the title, `conventional commits <https://www.conventionalcommits.org>`__ are preferred. Check the repo history!
|
||||
- **General Discussion**: Before addressing anything other than clear-cut bugs, start a discussion on the `official Open edX forum <https://discuss.openedx.org>`__. This facilitates reaching a consensus on a high-level solution.
|
||||
- **Pull Requests**: For changes to Tutor core or plugin-specific modifications, open a pull request on the `Tutor repository <https://github.com/overhangio/tutor/pulls>`__ or the corresponding plugin repository.
|
||||
- **Running Tests and Code Formatting**:
|
||||
- Ensure all tests pass by running ``make test``. This is mandatory for both Tutor core and plugin contributions.
|
||||
- If formatting tests fail, correct your code format using ``make format``.
|
||||
- **Changelog Entry**: Create a changelog entry for significant changes (excluding reformatting or documentation) by running ``make changelog-entry``. Edit the newly created file following the given formatting instructions. This applies to both Tutor core and plugin changes.
|
||||
- **Commit Messages**: Write clear Git commit titles and messages. Detail the rationale for your changes, the issue being addressed, and your solution. Include links to relevant forum discussions and describe your use case. Detailed explanations are valuable. For commit titles, follow `conventional commits <https://www.conventionalcommits.org>`__ guidelines.Additionally, if your pull request addresses an existing GitHub issue, include 'Close #XXX' in your commit message, where XXX is the issue number.
|
||||
|
||||
Releasing a new version
|
||||
-----------------------
|
||||
|
||||
When releasing a new version:
|
||||
|
||||
- **Version Number**: Update the version number in `__about__.py`. For detailed guidelines on version numbering, refer to the (versioning guidelines :ref:`versioning`).
|
||||
- **Changelog Compilation**: Compile all changelog entries using ``make changelog``.
|
||||
- **Git Commit for Release**: Use the format ``git commit -a -m "vX.Y.Z"`` to indicate the new version in the git commit title.
|
||||
|
||||
Happy hacking! ☘️
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user