mirror of
https://github.com/clearml/clearml-helm-charts
synced 2025-01-31 00:56:52 +00:00
64 lines
4.9 KiB
Markdown
64 lines
4.9 KiB
Markdown
# Guidelines for Contributing
|
|
|
|
:+1::tada: Firstly, we thank you for taking the time to contribute! :tada::+1:
|
|
|
|
Contribution comes in many forms:
|
|
* Reporting [issues](https://github.com/allegroai/clearml-helm-charts/issues) you've come upon
|
|
* Participating in issue discussions in the [issue tracker](https://github.com/allegroai/clearml-helm-charts/issues) and the [ClearML community slack space](https://joinslack.clear.ml)
|
|
* Suggesting new features or enhancements
|
|
* Implementing new features or fixing outstanding issues
|
|
|
|
The following is a set of guidelines for contributing to ClearML.
|
|
These are primarily guidelines, not rules.
|
|
Use your best judgment and feel free to propose changes to this document in a pull request.
|
|
|
|
## Reporting Issues
|
|
|
|
By following these guidelines, you help maintainers and the community understand your report, reproduce the behavior, and find related reports.
|
|
|
|
Before reporting an issue, please check whether it already appears [here](https://github.com/allegroai/clearml-helm-charts/issues).
|
|
If it does, join the on-going discussion instead.
|
|
|
|
**Note**: If you find a **Closed** issue that may be the same issue which you are currently experiencing,
|
|
then open a **New** issue and include a link to the original (Closed) issue in the body of your new one.
|
|
|
|
When reporting an issue, please include as much detail as possible: explain the problem and include additional details to help maintainers reproduce the problem:
|
|
|
|
* **Use a clear and descriptive title** for the issue to identify the problem.
|
|
* **Describe the exact steps necessary to reproduce the problem** in as much detail as possible. Please do not just summarize what you did. Make sure to explain how you did it.
|
|
* **Provide the specific environment setup.** Include the `pip freeze` output, specific environment variables, Python version, and other relevant information.
|
|
* **Provide specific examples to demonstrate the steps.** Include links to files or GitHub projects, or copy/paste snippets which you use in those examples.
|
|
* **If you are reporting any ClearML crash,** include a crash report with a stack trace from the operating system. Make sure to add the crash report in the issue and place it in a [code block](https://help.github.com/en/articles/getting-started-with-writing-and-formatting-on-github#multiple-lines),
|
|
a [file attachment](https://help.github.com/articles/file-attachments-on-issues-and-pull-requests/), or just put it in a [gist](https://gist.github.com/) (and provide link to that gist).
|
|
* **Describe the behavior you observed after following the steps** and the exact problem with that behavior.
|
|
* **Explain which behavior you expected to see and why.**
|
|
* **For Web-App issues, please include screenshots and animated GIFs** which recreate the described steps and clearly demonstrate the problem. You can use [LICEcap](https://www.cockos.com/licecap/) to record GIFs on macOS and Windows, and [silentcast](https://github.com/colinkeenan/silentcast) or [byzanz](https://github.com/threedaymonk/byzanz) on Linux.
|
|
|
|
## Suggesting New Features and Enhancements
|
|
|
|
By following these guidelines, you help maintainers and the community understand your suggestion and find related suggestions.
|
|
|
|
Enhancement suggestions are tracked as GitHub issues. After you determine which repository your enhancement suggestion is related to, create an issue on that repository and provide the following:
|
|
|
|
* **A clear and descriptive title** for the issue to identify the suggestion.
|
|
* **A step-by-step description of the suggested enhancement** in as much detail as possible.
|
|
* **Specific examples to demonstrate the steps.** Include copy/pasteable snippets which you use in those examples as [Markdown code blocks](https://help.github.com/articles/markdown-basics/#multiple-lines).
|
|
* **Describe the current behavior and explain which behavior you expected to see instead and why.**
|
|
* **Include screenshots or animated GIFs** which help you demonstrate the steps or point out the part of ClearML which the suggestion is related to. You can use [LICEcap](https://www.cockos.com/licecap/) to record GIFs on macOS and Windows, and [silentcast](https://github.com/colinkeenan/silentcast) or [byzanz](https://github.com/threedaymonk/byzanz) on Linux.
|
|
|
|
## Pull Requests
|
|
|
|
Before you submit a new PR:
|
|
|
|
* Verify the work you plan to merge addresses an existing [issue](https://github.com/allegroai/clearml-helm-charts/issues) (If not, open a new one)
|
|
* Check related discussions in the [ClearML slack community](https://joinslack.clear.ml) (or start your own discussion on the `#clearml-dev` channel)
|
|
* Check your branch with `helm lint`
|
|
* Update `version` in `Chart.yaml` according [semver](https://semver.org/) rules
|
|
* Substitute `annotations` section in `Chart.yaml` annotating implementations (useful for Artifecthub changelog)
|
|
* Update chart README using [helm-docs](https://github.com/norwoodj/helm-docs)
|
|
|
|
In your PR include:
|
|
|
|
* A reference to the issue it addresses
|
|
* A brief description of the approach you've taken for implementing
|