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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Show ESMValTool logo on PyPI webpage #2065

Merged
merged 1 commit into from May 25, 2023
Merged

Show ESMValTool logo on PyPI webpage #2065

merged 1 commit into from May 25, 2023

Conversation

valeriupredoi
Copy link
Contributor

@valeriupredoi valeriupredoi commented May 25, 2023

Description

Sister PR to Tool's ESMValGroup/ESMValTool#3185 - our nice and happy logo has disappeared off the PyPI pkg page, most probably since we don't link it from the RAW file on the README


Checklist

It is the responsibility of the author to make sure the pull request is ready to review. The icons indicate whether the item will be subject to the 馃洜 Technical or 馃И Scientific review.


To help with the number pull requests:

@valeriupredoi valeriupredoi added the documentation Improvements or additions to documentation label May 25, 2023
@valeriupredoi valeriupredoi added this to the v2.9.0 milestone May 25, 2023
@codecov
Copy link

codecov bot commented May 25, 2023

Codecov Report

Merging #2065 (3a90f20) into main (88827d6) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #2065   +/-   ##
=======================================
  Coverage   92.79%   92.79%           
=======================================
  Files         236      236           
  Lines       12484    12484           
=======================================
  Hits        11585    11585           
  Misses        899      899           

@bouweandela
Copy link
Member

@valeriupredoi Could you update the title to something that reads well in the changelog please?

@valeriupredoi valeriupredoi changed the title use RAW path to logo in README (PyPI pkg logo has poofed) use RAW path to logo image in README (ESMValTool logo at PyPI has disappeared) May 25, 2023
@valeriupredoi
Copy link
Contributor Author

@bouweandela better now? I love the word "poof" 馃槵

@bouweandela
Copy link
Member

How about something like 'Show ESMValTool logo on PyPI webpage'? 'RAW path' doesn't mean much to most of the people reading the log.

@valeriupredoi valeriupredoi changed the title use RAW path to logo image in README (ESMValTool logo at PyPI has disappeared) Show ESMValTool logo on PyPI webpage (title by Bouwe, contents by V) May 25, 2023
@valeriupredoi valeriupredoi changed the title Show ESMValTool logo on PyPI webpage (title by Bouwe, contents by V) Show ESMValTool logo on PyPI webpage May 25, 2023
@valeriupredoi
Copy link
Contributor Author

reason prevailed, but do check out the second to last title 馃ぃ

@bouweandela bouweandela merged commit dd3ff29 into main May 25, 2023
4 checks passed
@bouweandela bouweandela deleted the raw_logo_in_readme branch May 25, 2023 16:45
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

3 participants