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 test failure #303

Conversation

vincentarelbundock
Copy link
Collaborator

Useless PR just to trigger tests on main

@vincentarelbundock vincentarelbundock changed the title news increment to trigger tests and see failures Fix test failure Jan 5, 2025
@vincentarelbundock
Copy link
Collaborator Author

I'm going to merge this now, since this is such a minor change. We are wrapping a call to dir_delete() that is used to clean-up after the test in a try. The dir_delete() call errored on a hard to diagnose error related to temporary directories and testthat environment.

Frankly, I doubt this will bite us. It might at CRAN release time (unlikely, I think), but we can cross that bridge when we get to it.

@vincentarelbundock vincentarelbundock merged commit c50ebef into etiennebacher:main Jan 5, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant