search

ejwa / gitinspector

:bar_chart: The statistical analysis tool for git repositories
GNU General Public License v3.0
2.36k stars 327 forks source link

Setup npm package #138

Closed bartvanandel closed 7 years ago

bartvanandel commented 7 years ago

I've put some effort into turning this nice tool into an npm package. I've just published and tested it, works like a charm! Would you consider pulling the changes into master?

NB: I've added myself as a contributor for this, but I've set up the package to point to the original (your) github page, assuming you'd agree. If this is not desired, please let me know and I'll change it. If you'd like to be able to push to npm yourself, let me know as well so I can set this up.

adam-waldenberg commented 7 years ago

Hi @bartvanandel. Yes, we can add it to master. I personally don't use npm regularly , so it's better somebody else handles it. If you like, I can add you as the maintainer of this part.

bartvanandel commented 7 years ago

@adam-waldenberg, that sounds fine to me.

Note that I may have to update the package version number if a new npm release is required, at least in package.json. I'll try to keep those changes to a minimum.

bartvanandel commented 7 years ago

Any news about this pull request?

adam-waldenberg commented 7 years ago

Hi @bartvanandel . Will get to it as soon as I get time. In the future, I'll make sure to modify package.json when a new version is branhced. I'm planing a 0.4.5 release next. I'll let you handle the publishing when new versions get released.

bartvanandel commented 7 years ago

Okay, thanks, please inform me when the time is ripe.

bartvanandel commented 7 years ago

Changed a few things a few minutes ago.

adam-waldenberg commented 7 years ago

@bartvanandel Officially merged in master. I'll let you know when I have 0.4.5 ready. I'm planning to have it ready for release relatively soon.