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.

Sign up for GitHub

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

Jump to bottom

x/pkgsite: readme badges broken #42454

Closed
Delta456 opened this issue Nov 9, 2020 · 11 comments
Closed

x/pkgsite: readme badges broken #42454

Delta456 opened this issue Nov 9, 2020 · 11 comments
Labels
FrozenDueToAge NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. pkgsite
Milestone
pkgsite/unplanned

Comments

@Delta456
Copy link

Delta456 commented Nov 9, 2020 •
edited
Loading

What is the URL of the page with the issue?

https://pkg.go.dev/github.com/Delta456/box-cli-maker/v2

What is your user agent?

Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:82.0) Gecko/20100101 Firefox/82.0

Screenshot

image
image

What did you do?

Accessed the above pages and see they display everything properly.

What did you expect to see?

Badges are displayed properly.

What did you see instead?

README.md has broken badges.
@gopherbot gopherbot added this to the Unreleased milestone Nov 9, 2020
@seankhliao
Copy link
Member

you have a newline between your package documentation and the package declaration, go doc doesn't recognize it either

@Delta456
Copy link
Author

Delta456 commented Nov 9, 2020

There's one after the package declaration as for newline at the end of the file but I can see the first issue there.

@jba jba added the NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. label Nov 9, 2020
@jba jba modified the milestones: Unreleased, pkgsite/unplanned Nov 9, 2020
@julieqiu
Copy link
Member

julieqiu commented Nov 9, 2020

This might be related to #39297

@Delta456
Copy link
Author

I think the badges are broken because I have div align them in the center.

@Delta456
Copy link
Author

Delta456 commented Nov 10, 2020 •
edited
Loading

I have solved the displaying of doc.go issue but now the code isn't getting detected.

image

EDIT: do I have to tab my code lines?

EDIT 2: I have fixed it!

@Delta456 Delta456 changed the title x/pkgsite: doc.go and readme badges broken x/pkgsite: readme badges broken Nov 11, 2020
@Delta456
Copy link
Author

I have updated the OP as I have resolved doc.go display issue, thanks to @seankhliao. Now just the badges are broken.

@Delta456
Copy link
Author

Also shouldn't New function be placed under Functions sub heading? It is currently paced under Types.

image

@seankhliao
Copy link
Member

It counts as a constructor for the Box type

@Delta456
Copy link
Author

Ok

@jamalc jamalc self-assigned this Nov 18, 2020
@jamalc
Copy link

jamalc commented Nov 18, 2020

@Delta456 - this should now be fixed.

@jamalc jamalc closed this as completed Nov 18, 2020
@Delta456
Copy link
Author

@jamalc Thank You So Much!

@golang golang locked and limited conversation to collaborators Nov 19, 2021
@rsc rsc unassigned jamalc Jun 23, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
FrozenDueToAge NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. pkgsite
Projects
None yet
Milestone
pkgsite/unplanned
Development

No branches or pull requests
6 participants
@julieqiu @gopherbot @seankhliao @jamalc @jba @Delta456