Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs: managedFieldsManagers example in docs needs double-quotes #14324

Merged
merged 1 commit into from
Jul 6, 2023

Conversation

pgpx
Copy link
Contributor

@pgpx pgpx commented Jul 3, 2023

Fixes #14323

On https://argo-cd.readthedocs.io/en/stable/user-guide/diffing/#application-level-configuration the managedFieldsManagers example should have double-quoted the * wildcards (e.g. "*") because * is a YAML special character, and without this the examples cannot be parsed by ArgoCD (raising an error).

* is a Yaml special character and must be quoted, otherwise ArgoCD fails to parse and shows an error.

Signed-off-by: Paul Martin <paul.martin@gmail.com>
@pgpx pgpx changed the title fix(docs): Fix quotes for ignoreDifferences example fix(docs): managedFieldsManagers example in docs needs double-quotes Jul 3, 2023
@codecov
Copy link

codecov bot commented Jul 3, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (d7632df) 49.76% compared to head (e37d36b) 49.76%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master   #14324   +/-   ##
=======================================
  Coverage   49.76%   49.76%           
=======================================
  Files         261      261           
  Lines       44676    44676           
=======================================
  Hits        22232    22232           
  Misses      20257    20257           
  Partials     2187     2187           

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

Copy link
Member

@ishitasequeira ishitasequeira left a comment

Choose a reason for hiding this comment

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

Thanks @pgpx , Looks good!!

Copy link
Collaborator

@crenshaw-dev crenshaw-dev left a comment

Choose a reason for hiding this comment

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

Great catch, thank you @pgpx!

@crenshaw-dev crenshaw-dev changed the title fix(docs): managedFieldsManagers example in docs needs double-quotes docs: managedFieldsManagers example in docs needs double-quotes Jul 6, 2023
@crenshaw-dev
Copy link
Collaborator

/cherry-pick release-2.8

@crenshaw-dev
Copy link
Collaborator

/cherry-pick release-2.7

@crenshaw-dev
Copy link
Collaborator

/cherry-pick release-2.6

@crenshaw-dev
Copy link
Collaborator

/cherry-pick release-2.5

@crenshaw-dev crenshaw-dev merged commit ef8dae8 into argoproj:master Jul 6, 2023
31 checks passed
gcp-cherry-pick-bot bot pushed a commit that referenced this pull request Jul 6, 2023
* is a Yaml special character and must be quoted, otherwise ArgoCD fails to parse and shows an error.

Signed-off-by: Paul Martin <paul.martin@gmail.com>
gcp-cherry-pick-bot bot pushed a commit that referenced this pull request Jul 6, 2023
* is a Yaml special character and must be quoted, otherwise ArgoCD fails to parse and shows an error.

Signed-off-by: Paul Martin <paul.martin@gmail.com>
gcp-cherry-pick-bot bot pushed a commit that referenced this pull request Jul 6, 2023
* is a Yaml special character and must be quoted, otherwise ArgoCD fails to parse and shows an error.

Signed-off-by: Paul Martin <paul.martin@gmail.com>
gcp-cherry-pick-bot bot pushed a commit that referenced this pull request Jul 6, 2023
* is a Yaml special character and must be quoted, otherwise ArgoCD fails to parse and shows an error.

Signed-off-by: Paul Martin <paul.martin@gmail.com>
crenshaw-dev pushed a commit that referenced this pull request Jul 6, 2023
…) (#14371)

* is a Yaml special character and must be quoted, otherwise ArgoCD fails to parse and shows an error.

Signed-off-by: Paul Martin <paul.martin@gmail.com>
Co-authored-by: Paul Martin <paul.martin@gmail.com>
crenshaw-dev pushed a commit that referenced this pull request Jul 6, 2023
…) (#14372)

* is a Yaml special character and must be quoted, otherwise ArgoCD fails to parse and shows an error.

Signed-off-by: Paul Martin <paul.martin@gmail.com>
Co-authored-by: Paul Martin <paul.martin@gmail.com>
crenshaw-dev pushed a commit that referenced this pull request Jul 6, 2023
…) (#14374)

* is a Yaml special character and must be quoted, otherwise ArgoCD fails to parse and shows an error.

Signed-off-by: Paul Martin <paul.martin@gmail.com>
Co-authored-by: Paul Martin <paul.martin@gmail.com>
crenshaw-dev pushed a commit that referenced this pull request Jul 6, 2023
…) (#14373)

* is a Yaml special character and must be quoted, otherwise ArgoCD fails to parse and shows an error.

Signed-off-by: Paul Martin <paul.martin@gmail.com>
Co-authored-by: Paul Martin <paul.martin@gmail.com>
schakrad pushed a commit to schakrad/argo-cd that referenced this pull request Jul 24, 2023
…proj#14324) (argoproj#14372)

* is a Yaml special character and must be quoted, otherwise ArgoCD fails to parse and shows an error.

Signed-off-by: Paul Martin <paul.martin@gmail.com>
Co-authored-by: Paul Martin <paul.martin@gmail.com>
Signed-off-by: schakrad <58915923+schakrad@users.noreply.github.com>
yyzxw pushed a commit to yyzxw/argo-cd that referenced this pull request Aug 9, 2023
…proj#14324)

* is a Yaml special character and must be quoted, otherwise ArgoCD fails to parse and shows an error.

Signed-off-by: Paul Martin <paul.martin@gmail.com>
@suzaku suzaku mentioned this pull request Aug 15, 2023
tesla59 pushed a commit to tesla59/argo-cd that referenced this pull request Dec 16, 2023
…proj#14324)

* is a Yaml special character and must be quoted, otherwise ArgoCD fails to parse and shows an error.

Signed-off-by: Paul Martin <paul.martin@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

managedFieldsManagers example in docs needs double-quotes
4 participants