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

Move TestUtils to an extension #550

Closed
devmotion opened this issue Oct 24, 2023 · 1 comment
Closed

Move TestUtils to an extension #550

devmotion opened this issue Oct 24, 2023 · 1 comment

Comments

@devmotion
Copy link
Member

There's a push in the community towards cleaning up dependencies and moving optional features to extension, now that these are supported in Julia >= 1.9. In particular, test utilities are often moved to an extension with a weak dependency on Test (if possible). For instance, this has been done in AbstractFFTs and an open PR to Distributions will hopefully be merged soon.

I suggest we also make Test a weak dependency in DynamicPPL.

@mhauru
Copy link
Member

mhauru commented Jan 6, 2025

Closing since we'll likely go with a different design for refactoring tests, discussed in TuringLang/Turing.jl#2412.

@mhauru mhauru closed this as completed Jan 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants