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

Update quick-install-script.md #11403

Closed
wants to merge 1 commit into from
Closed

Conversation

tthebst
Copy link

@tthebst tthebst commented Apr 2, 2020

Hi Guys,

Changed password to Harbor12345. Now the password in the docs is consistent with the password set by the quick-install script.

Tim

@codecov
Copy link

codecov bot commented Apr 2, 2020

Codecov Report

Merging #11403 into master will decrease coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #11403      +/-   ##
==========================================
- Coverage   55.43%   55.42%   -0.01%     
==========================================
  Files         913      913              
  Lines       52908    52908              
  Branches     1780     1780              
==========================================
- Hits        29328    29326       -2     
  Misses      20297    20297              
- Partials     3283     3285       +2     
Impacted Files Coverage Δ
...ig/vulnerability/vulnerability-config.component.ts 45.85% <0.00%> (-1.92%) ⬇️
...r/harbor/src/portal/src/app/shared/shared.utils.ts 34.14% <0.00%> (+2.43%) ⬆️

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 c0246e2...9cc7f80. Read the comment docs.

Changed password to Harbor12345. Now the password is consistent with quick-install script.

Signed-off-by: Tim Gretler <gretler.tim@gmail.com>
@jonasrosland jonasrosland added the area/doc Issues to track documentation tasks label May 27, 2020
Copy link
Contributor

@wy65701436 wy65701436 left a comment

Choose a reason for hiding this comment

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

lgtm

@tthebst
Copy link
Author

tthebst commented Jul 31, 2020

I just saw that the docs aren't in this repo anymore. The docs moved to another repo. I will open a PR in that repo. @wy65701436

Copy link
Member

@ninjadq ninjadq left a comment

Choose a reason for hiding this comment

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

lgtm, but it conflicts with master. Could you rebase it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/doc Issues to track documentation tasks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants