forked from keptn/lifecycle-toolkit
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
docs: refactor contributing guide - general guidelines (keptn#1411)
Signed-off-by: Yash Pimple <[email protected]> Signed-off-by: Yash Pimple <[email protected]> Signed-off-by: Moritz Wiesinger <[email protected]> Co-authored-by: Meg McRoberts <[email protected]> Co-authored-by: Moritz Wiesinger <[email protected]>
- Loading branch information
1 parent
754b416
commit dd08c4b
Showing
2 changed files
with
49 additions
and
35 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
46 changes: 46 additions & 0 deletions
46
docs/content/en/contribute/docs/contribution-guidelines/_index.md
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,46 @@ | ||
--- | ||
title: Contribution Guidelines | ||
description: Guidelines for contributing towards Keptn Lifecycle Toolkit | ||
weight: 300 | ||
--- | ||
|
||
Before using the **Keptn Lifecycle Toolkit** | ||
as a contributor to the Kepth Lifecycle Toolkit repository, | ||
it is expected that you comply with the guidelines while | ||
making contributions towards the repository. | ||
|
||
## Guidelines for contributing | ||
|
||
* Always fork the repository then clone that fork to your local system | ||
rather than clone `main` directly. | ||
Keptn, like most open source projects, | ||
severely restricts who can push changes directly to the `main` branch | ||
to protect the integrity of the repository. | ||
* Keep your language clean and crisp. | ||
* Smaller PR's are easier to review and so generally get processed more quickly; | ||
when possible, chunk your work into smallish PR's. | ||
However, we recognize that documentation work sometimes requires larger PRs, | ||
such as when writing a whole new section or reorganizing existing files. | ||
* You may want to squash your commits before creating the final PR, | ||
to avoid conflicting commits. | ||
This is **not mandatory**; the maintainers will squash your commits | ||
during the merge when necessary. | ||
* Be sure that the description of the pull request itself | ||
is meaningful and clear. | ||
This helps reviewers understand each commit | ||
and provides a good history after the PR is merged. | ||
* If your PR is not reviewed in a timely fashion, | ||
feel free to post a gentle reminder to the `#keptn-docs` Slack channel. | ||
* Resolve review comments and suggestions promptly. | ||
|
||
If you see a problem and are unable to fix it yourself | ||
or have an idea for an enhancement, | ||
please create an issue on the GitHub repository. | ||
|
||
* Provide specific and detailed information about the problem | ||
and possible solutions to help others understand the issue. | ||
* When reporting a bug, provide a detailed list of steps to reproduce the bug. | ||
If possible, also attach screenshots that illustrate the bug. | ||
* If you want to do the work on an issue, | ||
include that information in your description of the issue | ||
or in a comment to the issue. |