This is an archive of the discontinued LLVM Phabricator instance.

[docs] Remove a link to an outdated Go docs
ClosedPublic

Authored by JohnTitor on Jun 2 2022, 4:50 AM.

Details

Summary

That link returns 404, we have bindings code on https://github.com/llvm/llvm-project/tree/main/llvm/bindings/go
but it seems we haven't published it and there are no docs yet.

Diff Detail

Event Timeline

JohnTitor created this revision.Jun 2 2022, 4:50 AM
Herald added a project: Restricted Project. · View Herald TranscriptJun 2 2022, 4:50 AM
JohnTitor requested review of this revision.Jun 2 2022, 4:50 AM
Herald added a project: Restricted Project. · View Herald TranscriptJun 2 2022, 4:50 AM
durin42 accepted this revision.Jun 2 2022, 10:47 AM

Looks fine to me, but I have no idea what it would take to get the docs published there either. Maybe that's an easy fix?

This revision is now accepted and ready to land.Jun 2 2022, 10:47 AM
lattner accepted this revision.Jun 2 2022, 1:03 PM
JohnTitor added a comment.EditedJun 3 2022, 7:48 AM

Looks fine to me, but I have no idea what it would take to get the docs published there either. Maybe that's an easy fix?

AFAIK it isn't _easy_. To publish the doc, we have to add our package to pkg.go.dev, but the current repo structure doesn't fit to do it. See https://pkg.go.dev/about#adding-a-package for more details about adding a package.
I think it needs some governance decision (and some reworks) so for now, removing the link makes some sense here.

This revision was landed with ongoing or failed builds.Jun 3 2022, 7:51 AM
This revision was automatically updated to reflect the committed changes.