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

Add npm-based changelog generator #17

Merged

Conversation

maniax89
Copy link
Contributor

I have a PR up to add in author URLs (similar to what is included with this PR)

lalitkapoor/github-changes#61

Until it gets merged, we can use my fork:

https://github.com/maniax89/github-changes/tree/author_url

@@ -65,6 +66,7 @@
"start": "webpack-dev-server --env dev",
"dist": "del dist; webpack --env prod",
"test": "karma start",
"test:watch": "npm test -- --no-single-run --auto-watch"
"test:watch": "npm test -- --no-single-run --auto-watch",
"generate-changelog": "node node_modules/github-changes -o sapientglobalmarkets -r react-querybuilder -a --use-commit-body --only-pulls --date-format=\"(YYYY/MM/DD)\""
Copy link
Contributor Author

Choose a reason for hiding this comment

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

not sure how to shorten this right now

@maniax89
Copy link
Contributor Author

not sure how well this would work when you had tons of history though - seems to be an all-or-nothing generation (no option to append to existing changelog) but it is OK for now

@maniax89
Copy link
Contributor Author

Addresses #16

@maniax89
Copy link
Contributor Author

fixed merge conflicts

@maniax89
Copy link
Contributor Author

any updates on this?

@pavanpodila pavanpodila merged commit 504178c into react-querybuilder:master Oct 18, 2016
@pavanpodila
Copy link
Contributor

Merged. I’ll make an NPM release later today. Thanks for all the great updates!

On Oct 18, 2016, at 12:12 PM, maniax89 [email protected] wrote:

any updates on this?


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub #17 (comment), or mute the thread https://github.com/notifications/unsubscribe-auth/AAJkroAc5-NGCAggn81e7j3vN_wV1Xm0ks5q1O_SgaJpZM4KWTnO.

@pavanpodila
Copy link
Contributor

Published v1.3 on NPM: https://www.npmjs.com/package/react-querybuilder

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