`vcr::vcr_test_path` issue from vcr v1.0.0

Question

`vcr::vcr_test_path` issue from vcr v1.0.0

Closed this issue 3 years ago · 7 comments

With the latest version of vcr just up to CRAN I unfortunately caused breaking changes for some packages depending on vcr. The function vcr_test_path is the issue. The change to that function worked for some, but it apparently has broken other packages.

We have a proposed fix at ropensci/vcr#236 Can you install that fork remotes::install_github("dpprdan/vcr@fix/test_path") and see if that fixes the problem?

Answer 1 · 2021-05-25T14:26:32.000Z

Thanks for informing me! I'm going to check it right now.

Answer 2 · 2021-05-25T17:16:58.000Z

Thanks for checking it! We are still discussing this to make sure we're finding the best solution. Wait a bit longer and I'll get back to you. I plan to submit a fixed version soon - so you may not have to submit a new version yourself. Apologies for this headache!

Answer 3 · 2021-05-28T17:10:10.000Z

Okay, I've checked it. Sorry it took so long, I couldn't hurry up. Now it works as it supposed to.

Answer 4 · 2021-05-29T13:20:14.000Z

Thanks, @sckott for solving it. I'dbe grateful if you could inform me that you have successfully uploaded the new version to CRAN by closing this issue so that I can deploy updated version of my package. I've made some changes to tests when figuring out in more detail how vcr works.

Answer 5 · 2021-05-30T13:25:05.000Z

Okay, I will do that.

Answer 6 · 2021-06-01T16:40:02.000Z

A new version of vcr is on CRAN https://cloud.r-project.org/web/packages/vcr/ - CRAN checks for your package should be fixed soon once they are using the new version of vcr.

Answer 7 · 2021-06-07T11:16:49.000Z

Once again -- many thanks for your help! And thanks for a great and very useful package :)