-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
WIP: add lerna changelog #23
Conversation
Code sucks but at least it prints out the labels now. |
Almost done but it's needs to be generalized (just want to get it working for babel) |
var org = process.env.GITHUB_ORG; | ||
var repo = process.env.GITHUB_REPO; | ||
|
||
var tags = [ |
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.
Just need to not hardcode this in
a32076c
to
eb6349d
Compare
Didn't get to updating the readme but I would do that as well Maybe a screenshot or example of what it looks like (or just link to the next babel changelog release when we do it). |
Can pr the one in the repo instead of my branch. |
This thread has been automatically locked because there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Fixes #18
So far, I'm just getting a list of commits that are either the merge commit or a commit to master directly.
Running:
GITHUB_AUTH=auth-token-here GITHUB_ORG=babel GITHUB_REPO=babel ./node_modules/.bin/lerna changelog
Right now we error if you pass more than 2 args so we'll either have to support multiple args, do more
process.env
stuff, or add a config file?