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 for dev testthat #229

Merged
merged 1 commit into from
Dec 27, 2024
Merged

Conversation

hadley
Copy link
Contributor

@hadley hadley commented Nov 7, 2024

You really should not be using internal functions since their interface can change without warning. I have made the expedient fix, but I'd recommend not using internal functions in the future.

You really should not be using internal functions since their interface can change without warning. I have made the expedient fix, but I'd recommend not using internal functions in the future.
@hadley hadley mentioned this pull request Nov 7, 2024
6 tasks
@bguiastr bguiastr merged commit 08f212d into UUPharmacometrics:master Dec 27, 2024
1 of 7 checks passed
@bguiastr
Copy link
Collaborator

bguiastr commented Dec 27, 2024

Thank you for the fix! I know that relying on internal functions from other packages is not recommended and I will look into an alternative as a long term fix.

Merry Christmas 🎄!

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.

2 participants