-
Notifications
You must be signed in to change notification settings - Fork 38.9k
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
Adds a simple release-note builder that scrapes the Github API for recent PRs #7616
Conversation
GOPATH=$PWD go get github.com/google/go-querystring | ||
GOPATH=$PWD go build release-notes.go | ||
|
||
./release-notes --laste-release-pr=<pr # of the previous release> |
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.
should be "last-release-pr"
exit 1 | ||
fi | ||
|
||
set -o nounset |
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.
please move this to the top and use [[ -z "${1:-}" ]];
in the above check.
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.
done.
Travis and shippable are both unhappy with your change, failing trying to install godeps. Maybe you need to bite the bullet now and get the dependencies into our godeps tree.... |
223228a
to
fce355f
Compare
@roberthbailey does this look good to merge? Shippable failure is a flake. |
Yes. LGTM. |
Adds a simple release-note builder that scrapes the Github API for recent PRs
@roberthbailey
Closes #6213
Possibly worth automating further, but this will help standardize and organize.