Skip option added

[ppitonak]

Skip option added with same behavior as in https://github.com/expressjs/morgan. Function to determine if logging is skipped, defaults to false. This function will be called as skip(req, res).

app.use(expressWinston.logger({
  transports: ...
  skip: function(req, res) { return res.statusCode < 400 }
}));

[floatingLomas]

LGTM, but @bithavoc or whoever else will have to merge it.

[bithavoc]

@floatingLomas you have been commiter since forever

[floatingLomas]

I was at one point. I don't currently have write access, or at least that's what the message is telling me...

[floatingLomas]

Ok, I looked it up. :P I had write permissions when this repo was under the firebaseco namespace because I was a part of that group; I'm pretty sure I lost those permissions whenever you moved it into your own namespace.

[bithavoc]

@floatingLomas fixed, pls ping me when you need npm push, actually I can give you publishing rights, what's your NPM user?

[floatingLomas]

@bithavoc, cool. Perhaps not surprisingly, it is floatinglomas. :)

I'll do some PR cleanup in the morning.

[floatingLomas]

@ppitonak, can you rebase the new stuff in bithavoc/express-winston's master onto this and re-push it, so I can merge it?

[ppitonak]

rebased

[ppitonak]

Thanks for merging, when do you plan to publish release?

[floatingLomas]

Soon as @bithavoc adds me for npm publishing. :)

[bithavoc]

In a few mins

[bithavoc]

@floatingLomas done, you're the man now.

[floatingLomas]

Awesomo. Published.