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

doc: Update installation.md #703

Merged
merged 2 commits into from May 16, 2024
Merged

Conversation

Ileriayo
Copy link
Contributor

Replace invalid GitHub url reference with the right one that points to the codebase on the master branch.

@Ileriayo Ileriayo force-pushed the patch-1 branch 2 times, most recently from c89e7b4 to 4db0926 Compare April 30, 2024 13:41
@@ -19,7 +19,7 @@ Argo CD is running. Don't worry, without any configuration, it will not start me
We also provide a Kustomize base in addition to the plain Kubernetes YAML
manifests. You can use it as remote base and create overlays with your
configuration on top of it. The remote base's URL is
`https://github.com/argoproj-labs/argocd-image-updater/manifests/base`
`https://github.com/argoproj-labs/argocd-image-updater/tree/master/manifests/base`

Copy link
Collaborator

Choose a reason for hiding this comment

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

Maybe we can use the stable version, to be consistent with the install.yaml later in this doc:
https://github.com/argoproj-labs/argocd-image-updater/tree/stable/manifests/base

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Correct!

@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 66.27%. Comparing base (7d93c7a) to head (7400bd5).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #703   +/-   ##
=======================================
  Coverage   66.27%   66.27%           
=======================================
  Files          22       22           
  Lines        2150     2150           
=======================================
  Hits         1425     1425           
  Misses        591      591           
  Partials      134      134           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@chengfang chengfang self-requested a review May 3, 2024 18:08
@chengfang
Copy link
Collaborator

Pls see comments in #533

@chengfang
Copy link
Collaborator

chengfang commented May 7, 2024

Pls see comments in #533

@Ileriayo Based on the discussion in #533 , do you agree this change is no longer needed?

We could append something like below to clarify a bit:

This directory can be viewed here

@Ileriayo
Copy link
Contributor Author

Ileriayo commented May 9, 2024

Correct! @chengfang

@pasha-codefresh
Copy link
Collaborator

pasha-codefresh commented May 14, 2024

LGTM

@pasha-codefresh pasha-codefresh added the documentation Improvements or additions to documentation label May 14, 2024
@pasha-codefresh pasha-codefresh changed the title Update installation.md doc: Update installation.md May 14, 2024
@pasha-codefresh
Copy link
Collaborator

@Ileriayo could you please fix DCO?

Signed-off-by: Ileriayo Adebiyi <ileriayoadebiyi@gmail.com>
@Ileriayo
Copy link
Contributor Author

Ileriayo commented May 16, 2024

@Ileriayo could you please fix DCO?

Done @pasha-codefresh

@Ileriayo Ileriayo reopened this May 16, 2024
@pasha-codefresh pasha-codefresh merged commit 68183af into argoproj-labs:master May 16, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants