git-rev

git-rev copied to clipboard

Doesn't escape quotes properly.

1

As the author doesn't seem to be active anymore, I have used `kulak-at`'s fix from this PR [https://github.com/tblobaum/git-rev/pull/13](https://github.com/tblobaum/git-rev/pull/13) and published it as a new package here: [https://www.npmjs.com/package/get-git-data](https://www.npmjs.com/package/get-git-data)

chardos

Doesn't work on Windows

3

Either that or I'm doing something wrong. No API is called with data in the functional parameter.

WestonThayer

Handle errors -- what if `git` is missing or IO fails?

4

Wrapping the exec call and ignoring errors is asking for trouble.

bugeats

Added a new function to get last commit message.

drajk

publish latest code

2

bump version and publish to npm for latest goodness to be available.

defunctzombie

Have to manually change current working directory

1

I currently use git-rev as follows: ```js process.chdir(__dirname) git.short(function (str) { // ... }) ``` But isn't the first line redundant? In the beginning of git-rev's `index.js` file, there is...

epsil

allow testing git repo outside of project root

2

I want to use this module to help report version of my software running in M$'s Azure. Azure is a pretty nice environment, but the `wwwroot` (where the code was...

bewest

Offer synchronous methods

10

Build scripts are a main use case for this lib. Build scripts are often dumb and not optimized for async use.

bugeats

git-rev short returns nothing when app started with pm2

2

``` coffee git.short (rev) -> console.log rev #empty ``` it's ubuntu 12.04

voronianski

ghost