Skip to content

adds helper script for release notes#662

Merged
r2d4 merged 2 commits intoGoogleContainerTools:masterfrom
balopat:improve_release_process
Jun 11, 2018
Merged

adds helper script for release notes#662
r2d4 merged 2 commits intoGoogleContainerTools:masterfrom
balopat:improve_release_process

Conversation

@balopat
Copy link
Copy Markdown
Contributor

@balopat balopat commented Jun 8, 2018

for our convenience:

$ ./hack/release_note.sh 
Collecting PR commits between v0.7.0 and master...
---------
* make: fix release path [#650](https://github.com/GoogleContainerTools/skaffold/pull/650)
* Remove fsnotify [#646](https://github.com/GoogleContainerTools/skaffold/pull/646)
* Fixing the licence [#652](https://github.com/GoogleContainerTools/skaffold/pull/652)
* Update go-containerregistry [#651](https://github.com/GoogleContainerTools/skaffold/pull/651)
* cloudbuild: publish skaffold images on commit and tag [#655](https://github.com/GoogleContainerTools/skaffold/pull/655)
* Asciidocs and refdocs tooling [#648](https://github.com/GoogleContainerTools/skaffold/pull/648)
* cloudbuild: increase timeout to 20m [#658](https://github.com/GoogleContainerTools/skaffold/pull/658)

@balopat balopat force-pushed the improve_release_process branch from 5215f95 to ad4e4f4 Compare June 8, 2018 23:18
@balopat
Copy link
Copy Markdown
Contributor Author

balopat commented Jun 8, 2018

hmmm...I'm not sure how we want to manage dependencies of utilities written in go. Should I just simple add it to vendor, is that the recommended approach?

@balopat balopat force-pushed the improve_release_process branch from ad4e4f4 to d6fd69d Compare June 11, 2018 18:15
Copy link
Copy Markdown
Contributor

@r2d4 r2d4 left a comment

Choose a reason for hiding this comment

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

one small thing

# a simple little utility to list PRs for release notes:
# Run ./hack/release_notes.sh --help for more info

go get github.com/google/go-github/github
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

shouldn't need this since its vendored

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

right, forgot about that. Good catch!

@r2d4 r2d4 merged commit 42f865f into GoogleContainerTools:master Jun 11, 2018
@balopat balopat deleted the improve_release_process branch June 11, 2018 21:44
@r2d4 r2d4 mentioned this pull request Jul 1, 2018
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.

2 participants