Skip to content

Make Argo CD naming consistent#694

Merged
alexmt merged 2 commits intoargoproj:masterfrom
TangramFlex:argo-cd-naming
Nov 5, 2018
Merged

Make Argo CD naming consistent#694
alexmt merged 2 commits intoargoproj:masterfrom
TangramFlex:argo-cd-naming

Conversation

@tedmiston
Copy link
Contributor

This small PR changes instances of ArgoCD to Argo CD throughout the repo, and also fixes a few small typos.

Note: There are a few *.pb.go that are generated and I believe should not have been edited, however I'm not sure where I should edit upstream from the generation process. Can you make a suggestion on that?


On the official Argo CD product page, docs, and throughout most of this repo, the name is spelled "Argo CD".

@CLAassistant
Copy link

CLAassistant commented Oct 17, 2018

CLA assistant check
All committers have signed the CLA.

@tedmiston
Copy link
Contributor Author

I've signed the CLA above, so I believe this PR is now ready for review.

Copy link
Collaborator

@alexmt alexmt left a comment

Choose a reason for hiding this comment

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

Thank you for making this cleanup, @tedmiston ! The *.pb.go files are generated using make codegen command (upstream files are *.pb and https://github.com/argoproj/argo-cd/blob/master/pkg/apis/application/v1alpha1/types.go) but it is fine to make changes manually.

You've missed the only one autogenerated file: https://github.com/argoproj/argo-cd/blob/master/server/settings/settings.pb.go#L9

@tedmiston
Copy link
Contributor Author

@alexmt Thank you for reviewing. I resolved the typo and also freshly rebased off of upstream master. Please let me know if any other revisions should be made.

@jessesuen
Copy link
Member

Can we hold off on this while we check with our guy in product marketing? I was a proponent of the single word ArgoCD syntax for SEO purposes.

@tedmiston
Copy link
Contributor Author

The single word for SEO makes sense.

FWIW, it was me browsing your product site first and seeing Argo CD and not ArgoCD everywhere, then seeing mixed usage in the codebase that motivated this PR.

[I'd also be happy to take this PR in the other direction if ArgoCD is the current preference.]

Sample Google query results:

  • "argo cd" site:argoproj.io ~= 15
  • "argocd" site:argoproj.io = 0
  • "argo cd" site:github.* ~= 1100
  • "argocd" site:github.* ~= 773

@tedmiston
Copy link
Contributor Author

Just wanted to touch base on this PR for canonicalizing the naming to see if any action is required on my end.

@jessesuen
Copy link
Member

Just wanted to touch base on this PR for canonicalizing the naming to see if any action is required on my end.

I've been overruled. Argo CD will be the correct usage going forward and not ArgoCD. Once the conflicts are resolved, it should be good to go.

@tedmiston
Copy link
Contributor Author

Thanks for checking @jessesuen and approving @alexmt.

I've rebased off master and resolved the conflicts. I believe this PR is good to go.

@codecov-io
Copy link

codecov-io commented Nov 5, 2018

Codecov Report

Merging #694 into master will increase coverage by 0.39%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #694      +/-   ##
==========================================
+ Coverage   22.99%   23.38%   +0.39%     
==========================================
  Files          42       42              
  Lines        6253     6349      +96     
==========================================
+ Hits         1438     1485      +47     
- Misses       4626     4668      +42     
- Partials      189      196       +7
Impacted Files Coverage Δ
util/session/sessionmanager.go 27.41% <ø> (ø) ⬆️
controller/sync.go 28.64% <ø> (ø) ⬆️
util/oidc/oidc.go 2.76% <ø> (ø) ⬆️
util/argo/argo.go 20.06% <ø> (ø) ⬆️
server/project/project.go 63.97% <ø> (ø) ⬆️
controller/state.go 2.89% <ø> (ø) ⬆️
util/kube/kube.go 24.72% <ø> (ø) ⬆️
server/server.go 29.19% <ø> (-1.07%) ⬇️
server/application/application.go 6.77% <0%> (-0.08%) ⬇️
... and 1 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1206926...e24602d. Read the comment docs.

@alexmt alexmt merged commit 34bb60f into argoproj:master Nov 5, 2018
@tedmiston tedmiston deleted the argo-cd-naming branch November 5, 2018 20:07
leoluz pushed a commit to leoluz/argo-cd that referenced this pull request Mar 13, 2025
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants