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 "description cut off" warning at end too #2129

Merged
merged 1 commit into from
Apr 7, 2022

Conversation

tianon
Copy link
Member

@tianon tianon commented Apr 7, 2022

This has to be slightly more clever than the warning at the beginning to try and avoid weird markdown rendering (so it looks reasonably clean) but it isn't too horrible.

Fixes docker/hub-feedback#2220

This has to be slightly more clever than the warning at the beginning to try and avoid weird markdown rendering (so it looks reasonably clean) but it isn't *too* horrible.
@tianon
Copy link
Member Author

tianon commented Apr 7, 2022

https://hub.docker.com/r/tianon/test (currently) shows what this looks like:

image
image

@tianon
Copy link
Member Author

tianon commented Apr 7, 2022

I hand-checked the markdown of the other few that are currently over the limit and they all appeared to be pretty reasonable. 👍

Reference:
$ wc -c */README.md | sort -n | tail -n11
  24738 nats-streaming/README.md
  26858 mariadb/README.md
  27981 mongo/README.md
  28408 postgres/README.md
  28537 php/README.md
  30901 nextcloud/README.md
  31379 eclipse-temurin/README.md
  35875 openjdk/README.md
  36940 tomcat/README.md
  37533 influxdb/README.md

@yosifkit yosifkit merged commit 68cbff9 into docker-library:master Apr 7, 2022
@yosifkit yosifkit deleted the cut-off-warning-at-eof branch April 7, 2022 17:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Make the readme cut-off hint more visible
2 participants