AEGEE / oms-core-first

[DEPRECATED] Core module of the online membership system of AEGEE.
Other
5 stars 5 forks source link

Added logging feature using bunyan (supersedes #21) #27

Closed stebenve86 closed 8 years ago

stebenve86 commented 8 years ago

I have created a new PR with the dev branch as the target for the merge, sorry for the mess...

linuxbandit commented 8 years ago

no mess, don't worry

linuxbandit commented 8 years ago

I have waited to integrate this because @ingomueller-net 's changeset is bigger and I think it is better to have yours on top of his (less things to change)

alternatively I didn't understand quite fully the awesomeness of PRs and I complicated the problem in my mind :)

ingomueller-net commented 8 years ago

Yes, I think what you suggest is a good idea. It will help me some work. I can help with the rebasing of #27 to avoid extra work for @stebenve86.

ingomueller-net commented 8 years ago

I rebased this PR, see PR #34.