This is an archive of the discontinued LLVM Phabricator instance.

[docs][clang] Fix a broken link on the APINotes doc
ClosedPublic

Authored by JohnTitor on May 28 2022, 10:00 AM.

Details

Summary

This patch replaces a link with a GitHub one.

Fixes llvm#55748

Diff Detail

Event Timeline

JohnTitor created this revision.May 28 2022, 10:00 AM
Herald added a project: Restricted Project. · View Herald TranscriptMay 28 2022, 10:00 AM
JohnTitor requested review of this revision.May 28 2022, 10:00 AM
Herald added a project: Restricted Project. · View Herald TranscriptMay 28 2022, 10:00 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript
JohnTitor edited the summary of this revision. (Show Details)May 28 2022, 10:01 AM
JohnTitor added a reviewer: compnerd.
compnerd accepted this revision.Jun 10 2022, 7:05 PM

It seems that other docs are following the GitHub URLs as well. I wish we could do the relative document link instead of embedding the URL, but if that is not possible, then this makes sense to do.

This revision is now accepted and ready to land.Jun 10 2022, 7:05 PM

It seems that other docs are following the GitHub URLs as well. I wish we could do the relative document link instead of embedding the URL, but if that is not possible, then this makes sense to do.

I couldn't find the URL serving that file and I'm not familiar with doing that. Let's use a GitHub URL for now.

This revision was automatically updated to reflect the committed changes.