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

Updates to 3.x branch #43

Merged
merged 7 commits into from Jul 6, 2018
Merged

Updates to 3.x branch #43

merged 7 commits into from Jul 6, 2018

Conversation

@mostlyjason
Copy link

@mostlyjason mostlyjason commented Jun 28, 2018

Fixing merge conflict from #42

@Shwetajain148
Copy link

@Shwetajain148 Shwetajain148 commented Jul 6, 2018

The PR changes looks fine to me and I have also tested it successfully. Merging it.

@Shwetajain148 Shwetajain148 merged commit 0432977 into loggly:3.x Jul 6, 2018
@Shwetajain148
Copy link

@Shwetajain148 Shwetajain148 commented Jul 6, 2018

Since the legacy stuff is still being used in our library, we can see the legacy message when running with winston 3.0. I need to work more on this to upgrade fully with winston 3.0.

Currently, I just tried providing the initial support so that our library doesn't break anymore with winston 3.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.