From 956490df464ea40bac007da2150a702f9de019d8 Mon Sep 17 00:00:00 2001 From: Maxime Guyot Date: Thu, 6 Sep 2018 20:12:03 +0200 Subject: [PATCH] Replace CLA by DCO in review guidelines (#7559) Signed-off-by: Maxime Guyot --- REVIEW_GUIDELINES.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/REVIEW_GUIDELINES.md b/REVIEW_GUIDELINES.md index 09d1e801d3..53d35c97a1 100644 --- a/REVIEW_GUIDELINES.md +++ b/REVIEW_GUIDELINES.md @@ -4,7 +4,7 @@ Anyone is welcome to review pull requests. Besides our [technical requirements]( ## Process -The process to get a pull request merged is fairly simple. First, all required tests need to pass and the contributor needs to have a signed CLA. See [Charts Testing](https://github.com/helm/charts/blob/master/test/README.md) for details on our CI system and how you can provide custom values for testing. If there is a problem with some part of the test, such as a timeout issue, please contact one of the charts repository maintainers by commenting `cc @helm/charts-maintainers`. +The process to get a pull request merged is fairly simple. First, all required tests need to pass and the contributor needs to have a signed [DCO](https://www.helm.sh/blog/helm-dco/index.html). See [Charts Testing](https://github.com/helm/charts/blob/master/test/README.md) for details on our CI system and how you can provide custom values for testing. If there is a problem with some part of the test, such as a timeout issue, please contact one of the charts repository maintainers by commenting `cc @helm/charts-maintainers`. The charts repository uses the OWNERS files to provide merge access. If a chart has an OWNERS file, an approver listed in that file can approve the pull request. If the chart does not have an OWNERS file, an approver in the OWNERS file at the root of the repository can approve the pull request.