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

Add tag website build text to README.md #2113

Merged

Conversation

postscript-dev
Copy link
Collaborator

No description provided.

Copy link
Collaborator

@piponazo piponazo left a comment

Choose a reason for hiding this comment

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

Thanks for the new piece of doc 👏

I made some comments, but they might be out of scope for this PR. Let me know if you are willing to address some of them, or otherwise I am fine with merging this as it is.

README.md Show resolved Hide resolved
README.md Show resolved Hide resolved
@codecov
Copy link

codecov bot commented Mar 3, 2022

Codecov Report

Merging #2113 (09e8c7b) into main (a5b6db1) will not change coverage.
The diff coverage is n/a.

❗ Current head 09e8c7b differs from pull request most recent head 3f15aaf. Consider uploading reports for the commit 3f15aaf to get more accurate results

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #2113   +/-   ##
=======================================
  Coverage   63.98%   63.98%           
=======================================
  Files          97       97           
  Lines       19084    19084           
  Branches     9750     9750           
=======================================
  Hits        12211    12211           
  Misses       4599     4599           
  Partials     2274     2274           

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 a5b6db1...3f15aaf. Read the comment docs.

Copy link
Collaborator

@piponazo piponazo left a comment

Choose a reason for hiding this comment

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

Thanks for taking care of the suggestions 👍

@postscript-dev postscript-dev merged commit 30a1609 into Exiv2:main Mar 3, 2022
@postscript-dev postscript-dev deleted the add_tag_website_build_text branch March 3, 2022 15:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants