Skip to content
This repository was archived by the owner on May 20, 2025. It is now read-only.

Conversation

wetheredge
Copy link
Contributor

I put it immediately after Serialize/Deserialize, since after was more common. I also standardized the existing derives to Deserialize, Serialize, Debug, … order, but I can drop that commit if you'd prefer.

Debug implementations were checked with cargo clippy -- -Wmissing_debug_implementations

@woodruffw
Copy link
Member

Cool, thank you!

Debug implementations were checked with cargo clippy -- -Wmissing_debug_implementations

Would you mind adding that to the lib.rs as well, so that it gets checks automatically on all clippy runs (including CI)? There are some existing allow/deny uses, so it should fit right in 🙂

@woodruffw woodruffw added the enhancement New feature or request label Mar 9, 2025
@woodruffw woodruffw merged commit 9527499 into zizmorcore:main Mar 10, 2025
4 checks passed
@woodruffw
Copy link
Member

Thanks @wetheredge!

@wetheredge wetheredge deleted the debug branch March 10, 2025 22:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants