Refactor argument name for clarity 🧹

Renamed the argument `comment_on_commit` to `comment` in `main` function of `aicodebot_action.py` and `test_review.py` for better readability and simplicity. This change should make the code more intuitive to understand.

Author: TechNickAI

aicodebot_action.py

@@ -8,7 +8,7 @@
 import json, os, subprocess, sys
 
 
-def main(comment_on_commit=True):
+def main(comment=True):
     """Run the AICodeBot action"""
     cli_runner = setup_cli()
     review_status, review_comments = review_code(cli_runner)
@@ -17,7 +17,7 @@ def main(comment_on_commit=True):
     else:
         exit_status = 0
 
-    if comment_on_commit:
+    if comment:
         comment_on_commit(review_status, review_comments)
     sys.exit(exit_status)
 

tests/test_review.py

@@ -16,4 +16,4 @@ def test_commit_review(tmp_path):
     assert "aicodebot.yaml" in os.getenv("AICODEBOT_CONFIG_FILE")
 
     # TODO: Mock the github client so we can test the comment on commit as well
-    aicodebot_action.main(comment_on_commit=False)
+    aicodebot_action.main(comment=False)