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

DRUP-801 Add forked AddPathPlugin and removed required patch #54

Merged
merged 2 commits into from
May 20, 2019

Conversation

mxr576
Copy link
Contributor

@mxr576 mxr576 commented May 16, 2019

README.md contained a reference to an outdated required patch (103)
for php-http/client-common:1.x. The composer.json contained the correct
reference (113). Although, that patch also became outdated because it
did not contain a bulletproof fix for the problem.
The issue has been properly fixed in the 2.x branch. This is the reason
why we forked the fixed version from the 2.x branch and with that
also removed all the overhead caused by conditional patching the
php-http/client-common dependency.

php-http/client-common#171

TODO:

README.md contained a reference to an outdated required patch (103)
for php-http/client-common:1.x. The composer.json contained the correct
reference (113). Although, that patch also became outdated because it
did not contain a bulletproof fix for the problem.
The issue has been properly fixed in the 2.x branch. This is the reason
why we forked the fixed version from the 2.x branch and with that
also removed all the overhead caused by conditional patching the
php-http/client-common dependency.

php-http/client-common#171
@googlebot googlebot added the cla: yes Indicates CLA has been signed label May 16, 2019
@codecov-io
Copy link

Codecov Report

Merging #54 into 2.x will decrease coverage by 0.08%.
The diff coverage is 77.77%.

Impacted file tree graph

@@             Coverage Diff              @@
##                2.x      #54      +/-   ##
============================================
- Coverage     91.34%   91.26%   -0.09%     
- Complexity     1717     1722       +5     
============================================
  Files           266      267       +1     
  Lines          3338     3353      +15     
============================================
+ Hits           3049     3060      +11     
- Misses          289      293       +4
Impacted Files Coverage Δ Complexity Δ
src/Client.php 97.82% <100%> (+0.02%) 42 <0> (ø) ⬇️
src/HttpClient/Utility/Builder.php 95.45% <50%> (-4.55%) 19 <0> (ø)
src/HttpClient/Plugin/AddPathPlugin.php 83.33% <83.33%> (ø) 5 <5> (?)

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 b70be36...71e03c2. Read the comment docs.

@mxr576
Copy link
Contributor Author

mxr576 commented May 16, 2019

@balazswmann FYI, this PR addresses the documentation issue that you found.

@mxr576 mxr576 changed the title Add forked AddPathPlugin and removed required patch DRUP-801 Add forked AddPathPlugin and removed required patch May 16, 2019
Copy link
Contributor

@Jaesin Jaesin left a comment

Choose a reason for hiding this comment

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

This looks good to me.

@mxr576 mxr576 merged commit 18a6357 into apigee:2.x May 20, 2019
mxr576 added a commit to mxr576/apigee-edge-drupal that referenced this pull request May 21, 2019
Jaesin pushed a commit to mxr576/apigee-edge-drupal that referenced this pull request May 23, 2019
mxr576 added a commit to mxr576/apigee-edge-drupal that referenced this pull request May 23, 2019
@Jaesin Jaesin added this to the 2.0.3 milestone Jun 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes Indicates CLA has been signed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants