Skip to content
New issue

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

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

Already on GitHub? # to your account

Document fix for unauthorized push #752

Merged
merged 4 commits into from
May 16, 2022
Merged

Conversation

zhouzi
Copy link
Contributor

@zhouzi zhouzi commented May 16, 2022

This PR adds the solution to unauthorized push to the README, as initially reported in #744.

@peaceiris
Copy link
Owner

Thanks! I will use the following URL instead of saving the image under Git's control.

https://user-images.githubusercontent.com/30958501/168609359-242f59a9-34c6-4272-9793-d6b2cc823837.png

@codecov
Copy link

codecov bot commented May 16, 2022

Codecov Report

Merging #752 (48d3b49) into main (cf3948a) will not change coverage.
The diff coverage is n/a.

❗ Current head 48d3b49 differs from pull request most recent head 874da69. Consider uploading reports for the commit 874da69 to get more accurate results

@@           Coverage Diff           @@
##             main     #752   +/-   ##
=======================================
  Coverage   61.45%   61.45%           
=======================================
  Files           4        4           
  Lines         275      275           
  Branches       45       45           
=======================================
  Hits          169      169           
  Misses        105      105           
  Partials        1        1           

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 cf3948a...874da69. Read the comment docs.

Copy link
Owner

@peaceiris peaceiris left a comment

Choose a reason for hiding this comment

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

LGTM!

@peaceiris
Copy link
Owner

#753 is another solution.

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants