Skip to content

Conversation

@francovp
Copy link
Contributor

Adds a new optional configuration parameter to specify a required reviewer username for merge requests. If set, the bot validates that the specified reviewer username in the BOT_GITLAB_MERGE_REQUEST_REVIEWER_USERNAME environment variable is present in the event data reviewers array before processing the merge request.

Adds a new optional configuration parameter to specify a required reviewer username for merge requests. If set, the bot validates that the specified reviewer in the `BOT_GITLAB_MERGE_REQUEST_REVIEWER_USERNAME` environment variable is present in the event data `reviewers` array before processing the merge request.
@francovp francovp force-pushed the feat-merge-request-only-with-reviewer branch from 66ad484 to 873d2b9 Compare May 17, 2025 06:45
@francovp
Copy link
Contributor Author

@coolbeevip hello, I will appreciate your review, please 😊

@coolbeevip
Copy link
Owner

coolbeevip commented May 26, 2025

Thank you for your contribution @francovp

I understand that this feature is designed to constrain the execution of AI Review, ensuring that AI Review only runs when the submitter selects a reviewer configured with BOT_GITLAB_MERGE_REQUEST_REVIEWER_USERNAME. This definitely helps in avoiding blind execution of AI Review.

Is my understanding correct?

@coolbeevip coolbeevip merged commit 53319e4 into coolbeevip:main May 26, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants