jedcunningham commented on code in PR #24288: URL: https://github.com/apache/airflow/pull/24288#discussion_r896082690
########## docs/helm-chart/index.rst: ########## @@ -119,10 +119,10 @@ To uninstall/delete the ``airflow`` deployment: The command removes all the Kubernetes components associated with the chart and deletes the release. -Installing the Chart with ArgoCD --------------------------------- +Installing the Chart with ArgoCD or FluxCD +------------------------------------------ -When installing the chart using ArgoCD, you MUST set the two following values, or your application +When installing the chart using ArgoCD or FluxCD, you MUST set the two following values, or your application Review Comment: ```suggestion When installing the chart using Argo CD or Flux, you MUST set the two following values, or your application ``` ########## docs/helm-chart/index.rst: ########## @@ -119,10 +119,10 @@ To uninstall/delete the ``airflow`` deployment: The command removes all the Kubernetes components associated with the chart and deletes the release. -Installing the Chart with ArgoCD --------------------------------- +Installing the Chart with ArgoCD or FluxCD +------------------------------------------ Review Comment: ```suggestion Installing the Chart with Argo CD or Flux ----------------------------------------- ``` Just from looking at their site, they seem to just use "Flux", not "FluxCD"? Also looks like it should be "Argo CD" instead too. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
