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

Port over Folklore config headers and json_encoding_options #293

Merged
merged 2 commits into from
Jun 5, 2019
Merged

Conversation

mfn
Copy link
Collaborator

@mfn mfn commented May 31, 2019

Fixes #287

The plan is to also back-port this to the v1 and cut a new release (cherry-picking the first commit in here).

@mfn mfn self-assigned this May 31, 2019
@mfn mfn requested a review from rebing May 31, 2019 20:54
@mfn mfn closed this Jun 3, 2019
@mfn mfn deleted the mfn-json branch June 3, 2019 20:13
@mfn mfn restored the mfn-json branch June 3, 2019 20:26
@mfn mfn reopened this Jun 3, 2019
@mfn mfn force-pushed the mfn-json branch 2 times, most recently from 9819a47 to 2ec267b Compare June 5, 2019 17:37
@codecov-io
Copy link

codecov-io commented Jun 5, 2019

Codecov Report

Merging #293 into master will increase coverage by 0.17%.
The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #293      +/-   ##
============================================
+ Coverage     51.78%   51.96%   +0.17%     
  Complexity      321      321              
============================================
  Files            31       31              
  Lines          1091     1095       +4     
============================================
+ Hits            565      569       +4     
  Misses          526      526
Impacted Files Coverage Δ Complexity Δ
src/Rebing/GraphQL/GraphQLController.php 68.42% <100%> (+2.7%) 15 <0> (ø) ⬇️
src/config/config.php 100% <100%> (ø) 0 <0> (ø) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 757fecb...2ec267b. Read the comment docs.

@mfn mfn merged commit 2ec267b into master Jun 5, 2019
@mfn mfn deleted the mfn-json branch June 5, 2019 19:27
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.

Unescaped unicode response
3 participants