DilumAluthge commented 3 years ago

Based on my comments in https://github.com/chriselrod/ThreadingUtilities.jl/issues/3 (specifically https://github.com/chriselrod/ThreadingUtilities.jl/issues/3#issuecomment-762670118, https://github.com/chriselrod/ThreadingUtilities.jl/issues/3#issuecomment-762670839, and https://github.com/chriselrod/ThreadingUtilities.jl/issues/3#issuecomment-762671041), I'm starting to become suspicious that the Aqua tests are actually responsible for the timeouts.

So this pull request moves the Aqua tests to a separate workflow.

DilumAluthge commented 3 years ago

@chriselrod Can you merge this once CI is green?

codecov-io commented 3 years ago

Codecov Report

Merging #15 (cb05cd5) into master (8044368) will not change coverage. The diff coverage is n/a.

@@            Coverage Diff            @@
##            master       #15   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            3         3           
  Lines          100       101    +1     
=========================================
+ Hits           100       101    +1

Impacted Files	Coverage Δ
src/threadtasks.jl	`100.00% <0.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 8044368...cb05cd5. Read the comment docs.

DilumAluthge commented 3 years ago

CI is green on this one too.

DilumAluthge commented 3 years ago

Ugh. Never mind. Even with this PR, I got a timeout: https://github.com/chriselrod/ThreadingUtilities.jl/runs/1726363081?check_suite_focus=true

JuliaSIMD / ThreadingUtilities.jl

Run the Aqua tests in a separate workflow #15

Codecov Report