I56 timeout

[joeHickson]

closes #56

• reset return_timings and max_execution_time defaults once testing is complete

[seabbs]

Could you please also increment the package version in the DESCRIPTION to 1.2.0 and add this feature to a new news section in NEWS.md. 1.1.0 just hit CRAN so need to make it clear this feature is not in that version.

[seabbs]

Thanks looks really great - this will be a great help! Just waiting for CI and then will merge.

[joeHickson]

I'm just waiting on a final run of canada to complete too

[joeHickson]

That failed, I'll need to look at it tomorrow. I'll revert to draft whilst I investigate.

[seabbs]

I see region_timings -> region_timing needing to be updated in the docs.

[joeHickson]

I see region_timings -> region_timing needing to be updated in the docs.
you caught me mid-push!

[codecov-commenter]

Codecov Report

Merging #63 into master will decrease coverage by 0.01%.
The diff coverage is 0.00%.

@@            Coverage Diff            @@
##           master     #63      +/-   ##
=========================================
- Coverage    2.01%   2.00%   -0.02%     
=========================================
  Files          20      20              
  Lines        3765    3788      +23     
=========================================
  Hits           76      76              
- Misses       3689    3712      +23     

Impacted Files	Coverage Δ
R/epinow.R	0.00% <0.00%> (ø)
R/summarise.R	0.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 45e18a3...28d443f. Read the comment docs.

[joeHickson]

package check is failing because episoon is not available on the windows build. @seabbs any ideas?

[seabbs]

None. That is one of our dev packages but there should be no recent changes to it. If it is just a windows failure I am happy to push to master as is. EpiSoon is only used as a suggest and in none of the core functionality.

[seabbs]

It looks like the issue is with a failure to install the vctrs 📦 . I think that is a dependency of a dependency and likely due to a CRAN update from the developers....

[seabbs]

Yup looks like recently pushed to CRAN: https://cran.r-project.org/web/packages/vctrs/index.html

[joeHickson]

Annoying that windows is the only one affected.

[seabbs]

Looks great. GitHub docs action is a great idea. Thanks for adding this.