Use `withr` in tests of `data_write()` #464

etiennebacher · 2023-10-08T11:10:07Z

Close #462

codecov · 2023-10-08T11:22:32Z

Codecov Report

Merging #464 (722ead3) into main (6c3bcf5) will not change coverage.
The diff coverage is n/a.

❗ Current head 722ead3 differs from pull request most recent head be7957b. Consider uploading reports for the commit be7957b to get more accurate results

@@           Coverage Diff           @@
##             main     #464   +/-   ##
=======================================
  Coverage   89.03%   89.03%           
=======================================
  Files          72       72           
  Lines        5383     5383           
=======================================
  Hits         4793     4793           
  Misses        590      590

IndrajeetPatil

Perfect! Thanks.

Feel free to run {styler} once and then merge.

Use withr in tests of data_write()

be7957b

etiennebacher requested a review from IndrajeetPatil October 8, 2023 11:24

IndrajeetPatil approved these changes Oct 8, 2023

View reviewed changes

styling

64d2f84

etiennebacher merged commit b2d3b1b into main Oct 8, 2023
14 checks passed

etiennebacher deleted the test-data-write-withr branch October 8, 2023 12:14

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Use `withr` in tests of `data_write()` #464

Use `withr` in tests of `data_write()` #464

etiennebacher commented Oct 8, 2023

codecov bot commented Oct 8, 2023

IndrajeetPatil left a comment

Use withr in tests of data_write() #464

Use withr in tests of data_write() #464

Conversation

etiennebacher commented Oct 8, 2023

codecov bot commented Oct 8, 2023

Codecov Report

IndrajeetPatil left a comment

Choose a reason for hiding this comment

Use `withr` in tests of `data_write()` #464

Use `withr` in tests of `data_write()` #464