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

Test against more recent Node versions #49

Merged
merged 1 commit into from
Nov 17, 2017
Merged

Test against more recent Node versions #49

merged 1 commit into from
Nov 17, 2017

Conversation

oliversalzburg
Copy link
Contributor

No description provided.

@oliversalzburg
Copy link
Contributor Author

@marcbachmann Are you a maintainer of this project? We opened #47 a while ago and there has been no activity since. We're considering a fork.

@marcbachmann
Copy link
Collaborator

I have permission to merge and publish but I'm not really using this module anymore.

@oliversalzburg
Copy link
Contributor Author

We recently included it into our web stack and would appreciate if changes were merged upstream to avoid further fragmentation.

@marcbachmann
Copy link
Collaborator

I'm using http://npm.im/pino everywhere. You might want to switch to that as it's pretty well maintained and backed by nearform. The api is quite similar to bunyan and also this module.

@marcbachmann marcbachmann merged commit 7a6628d into villadora:master Nov 17, 2017
@oliversalzburg
Copy link
Contributor Author

Thanks for the heads-up :D But our move to bunyan isn't even that far in the past and we're still building up our code around it. Moving to something new right now would probably not fit our budget.

@oliversalzburg oliversalzburg deleted the test/node-versions branch November 17, 2017 14:38
@lutangar
Copy link

lutangar commented Dec 8, 2017

Hey @villadora, it seems like @marcbachmann can't maintain express-bunyan-logger anymore, maybe you could get in touch with @oliversalzburg or just add him as a maintainer, since their team is still using this project on a daily basis.

@marcbachmann
Copy link
Collaborator

marcbachmann commented Dec 8, 2017

Looks like by now I had permission to add @oliversalzburg to the npm repo.
I just don't have any permission on github.
I could still merge stuff on here. But don't have much time to actively contribute.

@marcbachmann
Copy link
Collaborator

marcbachmann commented Dec 8, 2017

I'd say we should setup semantic release, so pushing something to npm gets much easier.

@oliversalzburg
Copy link
Contributor Author

We would be happy to maintain it. I'll wait for @villadora to give us a signal :)

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.

3 participants