pulumi / pulumi-kubernetes-operator

A Kubernetes Operator that automates the deployment of Pulumi Stacks
Apache License 2.0
221 stars 55 forks source link

Rewrite ReconcileStackSession out #355

Open squaremo opened 1 year ago

squaremo commented 1 year ago

The helper struct ReconcileStackSession makes engineering more difficult because it duplicates local variables, making it unclear which are in play, and obscuring side-effects. It doesn't do anything that couldn't better be done by funcs (or methods on *ReconcileStack), since it doesn't include any state that needs encapsulation.