Skip to content

Revert "Fix missing finalizers in the ArgoCD app"#175

Merged
simu merged 1 commit intomasterfrom
revert-additional-finalizers
Apr 23, 2026
Merged

Revert "Fix missing finalizers in the ArgoCD app"#175
simu merged 1 commit intomasterfrom
revert-additional-finalizers

Conversation

@simu
Copy link
Copy Markdown
Member

@simu simu commented Apr 22, 2026

With projectsyn/component-argocd#287 we shouldn't need to manage the dynamically added ArgoCD finalizers on the ArgoCD app anymore.

This reverts commit e676045.

Checklist

  • The PR has a meaningful title. It will be used to auto-generate the
    changelog.
    The PR has a meaningful description that sums up the change. It will be
    linked in the changelog.
  • PR contains a single logical change (to build a better changelog).
  • Categorize the PR by adding one of the labels:
    bug, enhancement, documentation, change, breaking, dependency
    as they show up in the changelog.
  • Link this PR to related issues or PRs.

With projectsyn/component-argocd#287 we
shouldn't need to manage the dynamically added ArgoCD finalizers on the
ArgoCD app anymore.

This reverts commit e676045.
@simu simu added bug Something isn't working bump:patch labels Apr 22, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 22, 2026

🚀 This PR has been released as v5.8.2

Triggering workflows Release

🛠️ Auto tagging enabled with label bump:patch

@simu simu marked this pull request as ready for review April 22, 2026 08:22
@simu simu requested a review from a team April 22, 2026 08:23
@simu simu merged commit 8e3c02d into master Apr 23, 2026
10 checks passed
@simu simu deleted the revert-additional-finalizers branch April 23, 2026 12:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working bump:patch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants