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

added ./ for broken url link #10623

Merged
merged 4 commits into from Sep 17, 2022
Merged

added ./ for broken url link #10623

merged 4 commits into from Sep 17, 2022

Conversation

RutviShukla
Copy link
Contributor

@RutviShukla RutviShukla commented Sep 16, 2022

I have modified the broken URL link that was giving the 404 error by adding ./at the beginning of the URL link.

fixes #10376

@codecov
Copy link

codecov bot commented Sep 16, 2022

Codecov Report

Base: 45.75% // Head: 45.75% // No change to project coverage 👍

Coverage data is based on head (a3d5402) compared to base (192bc50).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##           master   #10623   +/-   ##
=======================================
  Coverage   45.75%   45.75%           
=======================================
  Files         236      236           
  Lines       28527    28527           
=======================================
  Hits        13053    13053           
  Misses      13669    13669           
  Partials     1805     1805           
Impacted Files Coverage Δ
util/settings/settings.go 51.36% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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

Copy link
Collaborator

@leoluz leoluz left a comment

Choose a reason for hiding this comment

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

Tks a lot for the contribution.
Please check my comments

Comment on lines 147 to 151
Command for setting default namespace:

```bash
kubectl config set-context --current --namespace=argocd
```
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please move this section after line 139.
I would also suggest to change the text "Command for setting default namespace" to "First we need to set the current namespace to argocd running the following command:"

Signed-off-by: RutviShukla <rutvi.shukla@ufl.edu>
Signed-off-by: RutviShukla <rutvi.shukla@ufl.edu>
Signed-off-by: RutviShukla <rutvi.shukla@ufl.edu>
Signed-off-by: RutviShukla <rutvi.shukla@ufl.edu>
@leoluz leoluz merged commit 158e92c into argoproj:master Sep 17, 2022
ashutosh16 pushed a commit to ashutosh16/argo-cd that referenced this pull request Oct 7, 2022
* added ./ for broken url link

Signed-off-by: RutviShukla <rutvi.shukla@ufl.edu>

* added command for setting default namespace

Signed-off-by: RutviShukla <rutvi.shukla@ufl.edu>

* added command for setting default namespace

Signed-off-by: RutviShukla <rutvi.shukla@ufl.edu>

* relocated incommand related to setting default namespace

Signed-off-by: RutviShukla <rutvi.shukla@ufl.edu>

Signed-off-by: RutviShukla <rutvi.shukla@ufl.edu>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants