Skip to content

fix: versions were not being updated in the root composer.json file - #281

Merged
bcoe merged 4 commits into
masterfrom
fix-php-bug
Oct 9, 2019
Merged

fix: versions were not being updated in the root composer.json file#281
bcoe merged 4 commits into
masterfrom
fix-php-bug

Conversation

@bcoe

@bcoe bcoe commented Oct 8, 2019

Copy link
Copy Markdown

Maps have different access patterns than objects, so when we switched to using a new Map(), updates to the root composer file began failing.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Oct 8, 2019
@bcoe
bcoe requested review from chingor13 and dwsupplee October 8, 2019 01:13
@codecov

codecov Bot commented Oct 8, 2019

Copy link
Copy Markdown

Codecov Report

Merging #281 into master will increase coverage by 0.22%.
The diff coverage is 60%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #281      +/-   ##
==========================================
+ Coverage   81.45%   81.68%   +0.22%     
==========================================
  Files          24       24              
  Lines        2965     2964       -1     
  Branches      244      246       +2     
==========================================
+ Hits         2415     2421       +6     
+ Misses        538      531       -7     
  Partials       12       12
Impacted Files Coverage Δ
src/updaters/root-composer.ts 86.44% <60%> (+11.44%) ⬆️

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 ac1b907...3564d3d. Read the comment docs.

Comment thread __snapshots__/root-composer.js
@bcoe bcoe added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Oct 9, 2019
@kokoro-team kokoro-team removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Oct 9, 2019
@bcoe
bcoe merged commit 85f056b into master Oct 9, 2019
@chingor13
chingor13 deleted the fix-php-bug branch September 27, 2021 19:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants