Sangyeong Park

Results 5 comments of Sangyeong Park

@crenshaw-dev, I’ve addressed all requested changes, including adding docs and e2e tests. Could you please take another look when you have a chance? Thanks!

Hi @crenshaw-dev, Thanks for the follow-up. I've updated the PR to address the additional changes you requested. Please let me know what you think when you have a moment. Thanks!

> Looking good! Added a couple questions about tests. > > Mind adding a note to docs/operator-manual/application.yaml that these fields are available? No need to copy the full spec from...

> @emirot good catch! I proposed a fix here: [sangyeong01#1](https://github.com/sangyeong01/argo-cd/pull/1) Thanks for the fix! I tested this by deploying a local build . Before the fix, it was stuck in...

Thanks @crenshaw-dev ! I've merged your PR into my branch. The fix is now included in this PR.