Skip to content
This repository has been archived by the owner on Jan 30, 2020. It is now read-only.

Json #58

Merged
merged 9 commits into from
Apr 9, 2018
Merged

Json #58

merged 9 commits into from
Apr 9, 2018

Conversation

gpolaert
Copy link
Contributor

@gpolaert gpolaert commented Sep 7, 2016

Changelog:

  • Add a Json formatter to the library

@sandrokeil
Copy link

Cool, this solves #13 👍

Can you fix the error PHP Fatal error: Cannot declare class ZendTest\Log\Formatter\XmlTest, because the name is already in use in /home/travis/build/zendframework/zend-log/test/Formatter/XmlTest.php on line 255 please. See Travis for more details.

@gpolaert
Copy link
Contributor Author

gpolaert commented Sep 8, 2016

I'm fixing build issue. I've just set my env.

@gpolaert
Copy link
Contributor Author

gpolaert commented Sep 8, 2016

Sorry for the incremental PR :)
@sandrokeil, is it okay now?

@luads
Copy link

luads commented Jan 30, 2017

+1

It would be useful for our Logentries integration.

@gpolaert
Copy link
Contributor Author

@Hisamu :) Use Logmatic.io instead #troll-I-m-a-Logmatic-Evangelist ^^

@gpolaert
Copy link
Contributor Author

@sandrokeil When do you plan a release? Are you depending of the Zend master release?

@sandrokeil
Copy link

@gpolaert Sorry, I'm not a maintainer.

@weierophinney weierophinney added this to the 2.10.0 milestone May 17, 2017
weierophinney added a commit that referenced this pull request Apr 9, 2018
@weierophinney
Merge pull request #58 from logmatic/JSON
fba29a8 
Json
weierophinney added a commit that referenced this pull request Apr 9, 2018
@weierophinney
Adds CHANGELOG entry for #58
796bd4c 
Sets date for 2.10.0 release.
weierophinney added a commit that referenced this pull request Apr 9, 2018
@weierophinney
Merge branch 'feature/58' into develop
aa6593a 
Close #58
@weierophinney
Copy link
Member

Thanks, @gpolaert!

@weierophinney weierophinney merged commit 9f96fa3 into zendframework:master Apr 9, 2018
@samsonasik samsonasik mentioned this pull request Apr 9, 2018
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
2.10.0
Development

Successfully merging this pull request may close these issues.
5 participants
@gpolaert @sandrokeil @luads @weierophinney @froschdesign