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

Fix Timber for Error objects #45

Merged
merged 3 commits into from
Sep 26, 2018
Merged

Fix Timber for Error objects #45

merged 3 commits into from
Sep 26, 2018

Conversation

gunar
Copy link
Contributor

@gunar gunar commented Sep 26, 2018

I hope this finally does it.
If we used Lerna I could've done it all in one PR.
Now I've got to merge this first, then experiment on postman staging.

Not even patch-package was able to help me this time.
I tried to run experiments on staging based on patch-package but it seems it
has a problem with named packages (e.g. *@invisible/*logger).

image

@gunar gunar requested a review from kewitz September 26, 2018 04:09
@gunar gunar self-assigned this Sep 26, 2018
@gunar gunar removed the request for review from kewitz September 26, 2018 04:17
@gunar gunar added the 🐛 bug label Sep 26, 2018
@kewitz
Copy link
Contributor

kewitz commented Sep 26, 2018

@gunar you can test Postman by installing the library straight from this branch.

CHANGELOG.md Outdated
@@ -1,3 +1,7 @@
*v3.0.0*
> Internal: Serialize error objects
> Internal: Rename `event` key on-the-fly
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why major, is this a breaking change?

@gunar gunar changed the title Fix Timber for Error objects and the event key Fix Timber for Error objects Sep 26, 2018
@gunar gunar merged commit 0b225c2 into master Sep 26, 2018
@gunar gunar deleted the gcg/fix-timber branch September 26, 2018 21:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants