From f3e3b7c2f24007d1b1bb9d1ffbfadcc17d2f974b Mon Sep 17 00:00:00 2001 From: Evgenii Korolevskii <102794661+e-korolevskii@users.noreply.github.com> Date: Mon, 25 Jul 2022 12:29:26 +0200 Subject: [PATCH] Update docs/contributors.md Co-authored-by: Ivan <98037481+IvanZosimov@users.noreply.github.com> --- docs/contributors.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/docs/contributors.md b/docs/contributors.md index fbdcaca..5ca1e01 100644 --- a/docs/contributors.md +++ b/docs/contributors.md @@ -76,7 +76,7 @@ Adding or changing tests is an integral part of making a change to the code. The - CI will start automatically with some checks. Wait until the end of the execution and make sure that all checks passed successfully. If some checks fail, you can open them one by one, try to find the reason for failing and make changes to your code to resolve the problem - Maintainers will review your pull request -- If the maintainer asks for any changes, edit your changes, push, and ask for another review +- If a maintainer requests changes, first of all, try to think about his request critically and only after that implement and request another review - If your PR gets accepted, it will soon be merged into the main branch. But your contribution will take effect only after the release of a new version of the action > Sometimes maintainers reject pull requests and that's ok! Usually, along with rejection, we supply the reason for it. Nonetheless, we still really appreciate you taking the time to do it, and we don't take that lightly :heart: