ansible-community / awx-operator-helm

Helm chart for the AWX operator.
https://ansible-community.github.io/awx-operator-helm/
GNU General Public License v3.0
28 stars 8 forks source link

Make starter Chart.yaml the source of truth for all versions #25

Closed miles-w-3 closed 1 week ago

miles-w-3 commented 1 week ago

@oraNod Thanks for the review, updated based on your feedbacl. If you confirm which version would be appropriate I can update that as well

oraNod commented 1 week ago

@oraNod Thanks for the review, updated based on your feedbacl. If you confirm which version would be appropriate I can update that as well

@miles-w-3 Thanks for updating! I think we're almost there. Made a couple of nit changes and left a comment about printing the pre-stripped result. We probably only need the one print call there and could make it a little more informative.

miles-w-3 commented 1 week ago

@oraNod I didn't intend to leave either print in, removed both in my latest commit