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.

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)