Skip to content

Remove auto merge for develop updates#18

Merged
The0mikkel merged 4 commits intomainfrom
develop
Nov 16, 2025
Merged

Remove auto merge for develop updates#18
The0mikkel merged 4 commits intomainfrom
develop

Conversation

@The0mikkel
Copy link
Member

No description provided.

Merge main into develop to update the develop branch to the latest
version
Copilot AI review requested due to automatic review settings November 16, 2025 16:59
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR removes the auto-merge functionality from the develop-update workflow, requiring all PRs from main to develop to be manually reviewed and merged.

  • Removed the auto_merge input parameter from the workflow
  • Removed all auto-merge logic including hotfix detection, commenting, and warning steps
  • Updated documentation to reflect that PRs are now created but not automatically merged

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
README.md Updated documentation to remove references to the auto_merge input and described the new behavior where PRs are created but not automatically merged. However, outdated hotfix detection documentation remains.
.github/workflows/develop-update.yml Removed the auto_merge input parameter and all related workflow steps (hotfix detection, auto-merge logic, commenting, and warnings) cleanly and completely.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

README.md Outdated
Comment on lines 179 to 182
- **Hotfix detection**: When the latest commit on `main` was merged from a different branch (e.g., a hotfix), the workflow will:
- Skip auto-merge to allow manual review
- Create a PR from `main` to `develop` with the changes.
- Add a comment to the PR explaining the situation
- Create a workflow warning for visibility
Copy link

Copilot AI Nov 16, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The documentation still describes hotfix detection behavior (adding comments and creating workflow warnings) that has been removed from the workflow. Since the auto-merge functionality and all related hotfix detection logic have been removed, this section should be updated or removed to accurately reflect that the workflow now simply creates a PR from main to develop without any special hotfix handling.

Copilot uses AI. Check for mistakes.
@The0mikkel The0mikkel enabled auto-merge November 16, 2025 17:02
@The0mikkel The0mikkel disabled auto-merge November 16, 2025 17:03
@The0mikkel The0mikkel merged commit 0ea1a10 into main Nov 16, 2025
5 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.

1 participant