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

Fix: process.slurp ignores the timeout #2385

Merged
merged 5 commits into from
Nov 29, 2021
Merged

Fix: process.slurp ignores the timeout #2385

merged 5 commits into from
Nov 29, 2021

Conversation

fthomas
Copy link
Member

@fthomas fthomas commented Nov 29, 2021

No description provided.

@codecov
Copy link

codecov bot commented Nov 29, 2021

Codecov Report

Merging #2385 (4814e9f) into master (00cf8b5) will not change coverage.
The diff coverage is 100.00%.

❗ Current head 4814e9f differs from pull request most recent head b8bd5fd. Consider uploading reports for the commit b8bd5fd to get more accurate results
Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2385   +/-   ##
=======================================
  Coverage   80.43%   80.43%           
=======================================
  Files         139      139           
  Lines        2392     2392           
  Branches       44       44           
=======================================
  Hits         1924     1924           
  Misses        468      468           
Impacted Files Coverage Δ
.../main/scala/org/scalasteward/core/io/process.scala 100.00% <100.00%> (ø)

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 00cf8b5...b8bd5fd. Read the comment docs.

@fthomas fthomas added this to the 0.14.0 milestone Nov 29, 2021
@fthomas fthomas added the bug Something isn't working label Nov 29, 2021
@fthomas fthomas merged commit 90663e5 into master Nov 29, 2021
@fthomas fthomas deleted the topic/slurp-timeout branch December 8, 2021 07:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant