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

x/pkgsite: don't display README section if it is empty #40867

Closed
julieqiu opened this issue Aug 18, 2020 · 6 comments
Closed

x/pkgsite: don't display README section if it is empty #40867

julieqiu opened this issue Aug 18, 2020 · 6 comments
Labels
FrozenDueToAge help wanted NeedsFix The path to resolution is known, but the work has not been done. pkgsite Suggested Issues that may be good for new contributors looking for work to do.

Comments

@julieqiu
Copy link
Member

If a package doesn't have a README, don't display it.

What did you do?

Visited https://pkg.go.dev/github.com/gnuy/subdivx-get@v0.0.0-20200818145511-51d3e492f137?tab=overview

What did you expect to see?

No README section:

image

What did you see instead?

An empty README section:

image

@gopherbot gopherbot added this to the Unreleased milestone Aug 18, 2020
@julieqiu julieqiu added pkgsite NeedsFix The path to resolution is known, but the work has not been done. help wanted Suggested Issues that may be good for new contributors looking for work to do. labels Aug 18, 2020
@ironbatshashank
Copy link

Instead of displaying no readme section, how about we autogenerate a readme section? I think it'll be more helpful.

@julieqiu
Copy link
Member Author

@ironbatshashank - what are your thoughts on what that would look like?

@IamCathal
Copy link

I think if there was to be some auto generated content it would be a bit difficult:

Pros

  • There is some more content on the screen and it doesn't look sparse

Cons

  • Would have to pull stats from the repo like stars/language and description possibly? Getting a usable structure of information from this that would work for all repos without a readme would be difficult. If a repo didn't have a readme in the first place I don't think they'd have much else information to take add generate our own.
  • Maybe nitpicking but users would expect this readme to be the actual readme and if the generated version and the actual version weren't the same it could lead to some confusion.

I would opt for just a simple box of placeholder text saying something like "Looks like this project doesn't have a readme yet" or something similar. It requires minimal work to implement and can't break. I've not contributed to a golang project before but I'd be up for learning the process of contributing after completing the contributing guide.

@julieqiu
Copy link
Member Author

I would opt for just a simple box of placeholder text saying something like "Looks like this project doesn't have a readme yet" or something similar. It requires minimal work to implement and can't break.

Thanks for the input, @IamCathal! Let's go with that suggestion.

I've not contributed to a golang project before but I'd be up for learning the process of contributing after completing the contributing guide.

If you're interested in contributing this fix, let me know. Happy to help you get started and answer any questions in #pkgsite Slack.

https://github.com/golang/pkgsite/blob/master/CONTRIBUTING.md#getting-started is a good starting point.

@IamCathal
Copy link

I failed to get the DB setup about 4 times on my own and couldn't progress with this issue, thank you very much @rahulw for the help with the setup.

@gopherbot
Copy link

Change https://golang.org/cl/252997 mentions this issue: internal/frontend: do not display empty READMEs

@golang golang locked and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
FrozenDueToAge help wanted NeedsFix The path to resolution is known, but the work has not been done. pkgsite Suggested Issues that may be good for new contributors looking for work to do.
Projects
None yet
Development

No branches or pull requests

4 participants