- 
                Notifications
    You must be signed in to change notification settings 
- Fork 76
Add docs for fetching SBOMs #711
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
Conversation
| The latest updates on your projects. Learn more about Vercel for Git ↗︎ 
 | 
| @MaedahBatool the Vercel preview for this PR (and others) seems to be broken. Is this something we're aware of? @willdollman can you please add  | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
| 
 hey @willdollman is the Vercel issue solved or still not working? | 
b75b85b    to
    e83da88      
    Compare
  
    | @MaedahBatool Yes, these changes only apply to 5.9 onwards so I was waiting until that release before merging - it was released yesterday so I can now merge. Is there anything special I need to do? The notion page is WIP https://www.notion.so/sourcegraph/Docs-Versioning-88d31f1e103343d2bdbc30a11595b384 Yes, the link works if you add /docs 👍 Can we fix the preview link? Github doesn't send notifications if you edit someone's comment, so I only just saw your original message. | 
Add public-facing docs for fetching SBOMs using
src.Pull Request approval
Although pull request approval is not enforced for this repository in order to reduce friction, merging without a review will generate a ticket for the docs team to review your changes. So if possible, have your pull request approved before merging.
TODO