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

serde_test should be dev-dependency #93

Open
Mingun opened this issue May 20, 2023 · 0 comments
Open

serde_test should be dev-dependency #93

Mingun opened this issue May 20, 2023 · 0 comments

Comments

@Mingun
Copy link

Mingun commented May 20, 2023

If I'm not missing something, it is required only for tests, so it should be in the [dev-dependencies] section

It is not required to made serde_test optional in dev-dependencies. It will be used only if some your code will depend on it, and this code will be included only when you enable the serde_support feature during tests. So if you run

cargo test --no-default-features

the serde_test dependency will not compiled (and even downloaded if I remember correctly) because you have no code that uses it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant