Skip to content

Enhance diff check logic in develop-update#6

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

Enhance diff check logic in develop-update#6
The0mikkel merged 2 commits intomainfrom
develop

Conversation

@The0mikkel
Copy link
Member

No description provided.

@The0mikkel The0mikkel self-assigned this Nov 16, 2025
Copilot AI review requested due to automatic review settings November 16, 2025 15:34
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 enhances the diff detection logic in the develop-update workflow by switching from content-based comparison to commit-based comparison, making it more robust for tracking changes that need to be merged from main to develop.

Key Changes:

  • Replaced git diff with git rev-list --count to check for commits in main that aren't in develop
  • Updated documentation to accurately reflect the new commit-based detection logic
  • Removed trailing whitespace for code cleanliness

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
.github/workflows/develop-update.yml Enhanced diff check logic to use git rev-list --count instead of git diff, added explanatory comments, and removed trailing whitespace
README.md Updated documentation to describe the new commit-based detection approach instead of general "merge detection"

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

@The0mikkel The0mikkel merged commit fc85082 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