Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Automated Merge Conflict Detection and Notification #746

Merged
merged 1 commit into from
Mar 8, 2024
Merged

Automated Merge Conflict Detection and Notification #746

merged 1 commit into from
Mar 8, 2024

Conversation

konavivekramakrishna
Copy link
Contributor

@konavivekramakrishna konavivekramakrishna commented Mar 4, 2024

Description:

Fixes : #745

This pull request proposes the implementation of automation to enhance our code review process by automatically detecting merge conflicts and notifying authors promptly. Currently, identifying merge conflicts during code review poses a challenge, often leading to delays and inefficiencies.

With this automation, when a merge conflict occurs, a label will be applied to the pull request, and a comment will be posted, specifically mentioning the author. This notification will advise the author to rebase their branch against the base branch to resolve the conflict efficiently. By implementing this solution, we aim to streamline our code review process and ensure smoother collaboration among team members.

Images for pr after detecting Conflicts
image
image

@konavivekramakrishna
Copy link
Contributor Author

@mozzy11 @CalebSLane

Could you please review the PR?

To proceed with merging, please add the "merge conflict" label and ensure that GITHUB_TOKEN is included in the GitHub variables.

@mozzy11
Copy link
Collaborator

mozzy11 commented Mar 8, 2024

@konavivekramakrishna i guess the secrest.GITHUB_TOKEN is auto generated by GitHub . no need to create it .https://docs.github.com/en/actions/security-guides/automatic-token-authentication

@mozzy11 mozzy11 merged commit c72a339 into I-TECH-UW:develop_3x Mar 8, 2024
2 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.

Add Automation to Merge Conflicts Detection
2 participants