Skip to content

chore: various improvements - #56

Merged
porridge merged 10 commits into
mainfrom
porridge/review-simple
Apr 13, 2026
Merged

chore: various improvements#56
porridge merged 10 commits into
mainfrom
porridge/review-simple

Conversation

@porridge

@porridgeporridge commented Feb 18, 2026

Copy link
Copy Markdown
Contributor

Please review commit-by commit.
Each represents a self-contained change and each of them should be merge-worthy as is.

@mclasmeiermclasmeier left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Just few minor comments. 👍

Comment threadinternal/deployer/operator.go Outdated
Comment threadinternal/deployer/deploy_via_helm.go Outdated
These just repeat what is obvious from the line which follows.
These are likely not needed.
If they are, synchronization should not be time-based.
@porridge
porridgeforce-pushed the porridge/review-simple branch from d7e8794 to 03a144fCompareApril 13, 2026 07:03
@porridge
porridge marked this pull request as ready for review April 13, 2026 07:03
@porridge
porridgeforce-pushed the porridge/review-simple branch from 03a144f to 9d6b88cCompareApril 13, 2026 07:09
Comment threadinternal/deployer/deploy_via_operator.go
Comment threadinternal/deployer/deploy_via_operator.go Outdated
@porridge
porridge merged commit 24b0c54 into mainApr 13, 2026
3 checks passed
@porridge
porridge deleted the porridge/review-simple branch April 14, 2026 08:12
Sign up for freeto 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.

2 participants

@porridge@mclasmeier