Skip to content

Return replaced/patched files when creating PR and don't send service message#1884

Open
APErebus wants to merge 4 commits intomainfrom
ap/return-files-for-pr
Open

Return replaced/patched files when creating PR and don't send service message#1884
APErebus wants to merge 4 commits intomainfrom
ap/return-files-for-pr

Conversation

@APErebus
Copy link
Copy Markdown
Contributor

@APErebus APErebus commented Apr 14, 2026

Related Server PR: https://github.com/OctopusDeploy/OctopusDeploy/pull/42189

Related to MD-1767

When creating a pull request, we no longer want to send the argocd-files-updated service message. This is because this message causes an update of the tracked git state in Octopus Server.
For a pull request, this git state should only change when the PR is merged.

We now don't send this service message when the git commit parameters IsPr is true.
We also include the replaced & patched files in new output variables, which are then captured in Octopus Server and attached to the pull request tracking entity.

Comment thread source/Calamari.Tests/ArgoCD/ArgoCDFilesUpdatedReporterTests.cs Outdated
@APErebus APErebus changed the title Return replaced/patched files when creating PR Return replaced/patched files when creating PR and don't send service message Apr 16, 2026
@APErebus APErebus requested a review from flin-8 April 16, 2026 04:01
@APErebus APErebus marked this pull request as ready for review April 16, 2026 04:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants