Skip to content

build(package update): update version of concurrently #34

Open
mdhitchcock wants to merge 2 commits intokentcdodds:mainfrom
mdhitchcock:update-concurrently
Open

build(package update): update version of concurrently #34
mdhitchcock wants to merge 2 commits intokentcdodds:mainfrom
mdhitchcock:update-concurrently

Conversation

@mdhitchcock
Copy link
Copy Markdown

Update version of concurrently away from buggy 3.6.1 and to more current ^4.1

The version of concurrently that the current package.json locks in is old and has a couple serious
bugs (https://github.com/kimmobrunfeldt/concurrently/issues/147 and
https://github.com/kimmobrunfeldt/concurrently/issues/193). One effect is that any keystroke in a
terminal running a package script using nps-utils will crash and leave processes running. This
change is just to bump the version of concurrently from ^3.4.0 (which will install 3.6.1) to version
^4.1.0

Mike Hitchcock added 2 commits September 26, 2019 10:41
… 3.6.1 and to more current ^4.1

The version of concurrently that the current package.json locks in is old and has a couple serious
bugs (https://github.com/kimmobrunfeldt/concurrently/issues/147 and
https://github.com/kimmobrunfeldt/concurrently/issues/193). One effect is that any keystroke in a
terminal running a package script using nps-utils will crash and leave processes running. This
change is just to bump the version of concurrently from ^3.4.0 (which will install 3.6.1) to version
^4.1.0
…on of concurrently dependency

The previous commit updated the version of concurrently because the version being used has serious
bugs; but the change failed validation because the snapshot files needed to be updated to reflect
the new file layout of the concurrently package
@codecov
Copy link
Copy Markdown

codecov bot commented Dec 4, 2019

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (main@b20c1db). Learn more about missing BASE report.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff            @@
##             main       #34   +/-   ##
========================================
  Coverage        ?   100.00%           
========================================
  Files           ?         1           
  Lines           ?        79           
  Branches        ?         0           
========================================
  Hits            ?        79           
  Misses          ?         0           
  Partials        ?         0           

Continue to review full report in Codecov by Sentry.

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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.

1 participant