Skip to content

cargo test -r fails due to cfg(debug_assertions) #1962

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.

Sign up for GitHub

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

Jump to bottom
Closed
joshtriplett opened this issue Apr 21, 2025 · 0 comments · Fixed by #1963
Closed

cargo test -r fails due to cfg(debug_assertions) #1962

joshtriplett opened this issue Apr 21, 2025 · 0 comments · Fixed by #1963

Comments

@joshtriplett
Copy link
Contributor

Attempting to run cargo test -r failed in gix-url due to missing gix_url::testing::TestUrlExtension, which only exists with cfg(debug_assertions). I think this should be using cfg(test) instead.
@Byron Byron linked a pull request Apr 21, 2025 that will close this issue
Switch to zlib-rs by default and drop other zlib backends #1963
Merged
Byron added a commit to joshtriplett/gitoxide that referenced this issue Apr 21, 2025
@Byron
Allow testing::TestUrlExtension in release builds for consistency (G…
633f0a8 
…itoxideLabs#1962)

The module is still hidden from the documentation, and those who discover
it anyway will be informed that it's not for general use.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Switch to zlib-rs by default and drop other zlib backends joshtriplett/gitoxide
1 participant
@joshtriplett