allow PR reviews to be passive

ref cpp-linter/cpp-linter-action#243

Author: 2bndy5

cpp_linter/cli.py

@@ -67,6 +67,8 @@ class Args(UserDict):
     ignore_tidy: str = ""
     #: See :std:option:`--ignore-format`.
     ignore_format: str = ""
+    #: See :std:option:`--passive-reviews`.
+    passive_reviews: bool = False
 
 
 _parser_args: Dict[Sequence[str], Any] = {}
@@ -343,6 +345,12 @@ class Args(UserDict):
 
 Defaults to ``%(default)s``.""",
 )
+_parser_args[("-R", "--passive-reviews")] = dict(
+    default="false",
+    type=lambda input: input.lower() == "true",
+    help="""Set to ``true`` to prevent Pull Request
+reviews from requesting or approving changes.""",
+)
 
 
 def _parse_jobs(val: str) -> Optional[int]:

cpp_linter/rest_api/github_api.py

@@ -212,6 +212,7 @@ def post_feedback(
                 tidy_review=args.tidy_review,
                 format_review=args.format_review,
                 no_lgtm=args.no_lgtm,
+                passive_reviews=args.passive_reviews,
             )
 
     def make_annotations(
@@ -346,6 +347,7 @@ def post_review(
         tidy_review: bool,
         format_review: bool,
         no_lgtm: bool,
+        passive_reviews: bool,
     ):
         url = f"{self.api_url}/repos/{self.repo}/pulls/{self.pull_request}"
         response = self.api_request(url=url)
@@ -393,6 +395,8 @@ def post_review(
                 return
             body += "\nGreat job! :tada:"
             event = "APPROVE"
+        if passive_reviews:
+            event = "COMMENT"
         body += USER_OUTREACH
         payload = {
             "body": body,

tests/reviews/test_pr_review.py

@@ -29,6 +29,7 @@
     summary_only=False,
     no_lgtm=False,
     num_workers=None,
+    is_passive=False,
 )
 
 
@@ -56,6 +57,7 @@ def mk_param_set(**kwargs) -> OrderedDict:
         tuple(mk_param_set(tidy_review=True, changes=1).values()),
         tuple(mk_param_set(tidy_review=True, changes=0).values()),
         tuple(mk_param_set(tidy_review=True, changes=0, summary_only=True).values()),
+        tuple(mk_param_set(is_passive=True).values()),
     ],
     ids=[
         "draft",
@@ -68,6 +70,7 @@ def mk_param_set(**kwargs) -> OrderedDict:
         "lines_added",
         "all_lines",
         "summary_only",
+        "passive",
     ],
 )
 def test_post_review(
@@ -83,6 +86,7 @@ def test_post_review(
     summary_only: bool,
     no_lgtm: bool,
     num_workers: int,
+    is_passive: bool,
 ):
     """A mock test of posting PR reviews"""
     # patch env vars
@@ -162,6 +166,7 @@ def test_post_review(
         args.thread_comments = "false"
         args.no_lgtm = no_lgtm
         args.file_annotations = False
+        args.passive_reviews = is_passive
 
         capture_clang_tools_output(files, args=args)
         if not force_approved:
@@ -211,7 +216,10 @@ def test_post_review(
             if force_approved:
                 assert json_payload["event"] == "APPROVE"
             else:
-                assert json_payload["event"] == "REQUEST_CHANGES"
+                if is_passive:
+                    assert json_payload["event"] == "COMMENT"
+                else:
+                    assert json_payload["event"] == "REQUEST_CHANGES"
 
             # save the body of the review json for manual inspection
             assert hasattr(last_request, "text")