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

docs: describe build process #109

Merged
merged 4 commits into from
Jan 9, 2023
Merged

Conversation

jfhovinne
Copy link
Contributor

I would suggest to document the release process, and in particular how to reproduce the builds locally.

README.md Outdated
@@ -59,6 +59,13 @@ $ go install github.com/google/osv-scanner/cmd/osv-scanner@v1

This requires Go 1.18+ to be installed.

### Releasing
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it would be better to title this "Building", and frame it from that POV - so start with explaining that you can build from source using goreleaser build --single-target, and then mention that version builds can be reproduced by checking out the specific tag and doing a build.

Is probably also worth linking to the goreleaser build docs

@jfhovinne jfhovinne changed the title docs: describe release process docs: describe build process Jan 2, 2023
Copy link
Collaborator

@G-Rath G-Rath left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just got one small comment about a word but after that's applied LGTM 👍

README.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@oliverchang oliverchang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! The README.md is getting big enough that I think we're gonna want to have a CONTRIBUTING.md doc soon where this can go into.

We can move this there.

jfhovinne and others added 3 commits January 6, 2023 12:09
@another-rex
Copy link
Collaborator

/gcbrun

@another-rex another-rex merged commit ef29ce4 into google:main Jan 9, 2023
julieqiu pushed a commit to julieqiu/osv-scanner that referenced this pull request May 2, 2023
* docs: describe release process

* docs: describe build process

* docs: typo

Co-authored-by: Gareth Jones <Jones258@Gmail.com>

Co-authored-by: Gareth Jones <Jones258@Gmail.com>
Co-authored-by: Rex P <106129829+another-rex@users.noreply.github.com>
julieqiu pushed a commit to julieqiu/osv-scanner that referenced this pull request May 2, 2023
* docs: describe release process

* docs: describe build process

* docs: typo

Co-authored-by: Gareth Jones <Jones258@Gmail.com>

Co-authored-by: Gareth Jones <Jones258@Gmail.com>
Co-authored-by: Rex P <106129829+another-rex@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants