Skip to content

Commit 39c7122

Browse files
committed
Added requested changes and extra steps for the Sprint Board
1 parent 009cc19 commit 39c7122

File tree

1 file changed

+7
-5
lines changed

1 file changed

+7
-5
lines changed

CONTRIBUTING.md

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -31,13 +31,15 @@ This project uses ZenHub for tracking of issues and roadmap.
3131
## Reporting an Issue
3232

3333
To report an issue, or to suggest an idea for a change, open an [issue](../../issues/new). It is best to check our existing [issues](../../issues) first to see if a similar one has already been opened and discussed.
34-
## Getting Feedback on your contribution
34+
## Getting Feedback on Your Contribution
3535

3636
When creating a PR for one of our projects it is important to get your work reviewed by other members of the team. Below are some steps to getting your work reviewed. <br>
37-
* Assign a Reviewer from the list of reviewers on the project
38-
* Get a `lgtm` label for your PR
39-
* Assign an Approver to your PR
40-
* Based on the quality of your work it will be approved and merged or changes will be requested
37+
* Assign a Reviewer from the list of reviewers on the project.
38+
* Move the linked issue(s) from the `In Progress` column to `Ready For Review` column on the [Project Codeflare Sprint Board](https://github.com/orgs/project-codeflare/projects/8/views/3).
39+
* Once the PR is reviewed the reviewer may request changes to be made, or if they are happy with the PR they will apply an `lgtm` label.
40+
* You can now move the linked issue(s) to the `In Review` column.
41+
* When the `lgtm` label has been applied you can assign an Approver to your PR.
42+
* The approver may request more changes, or if they are happy with the PR they will apply the `approved` label. After this the PR will be merged.
4143

4244
### Reviewers and Approvers for each project
4345
* [MCAD](https://github.com/project-codeflare/multi-cluster-app-dispatcher/blob/main/OWNERS)

0 commit comments

Comments
 (0)