Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

Hide KeyedHashDB behind #[cfg(test)] #10387

Closed
ordian opened this issue Feb 20, 2019 · 1 comment
Closed

Hide KeyedHashDB behind #[cfg(test)] #10387

ordian opened this issue Feb 20, 2019 · 1 comment
Labels
F6-refactor 📚 Code needs refactoring. P7-nicetohave 🐕 Issue is worth doing eventually.

Comments

@ordian
Copy link
Collaborator

ordian commented Feb 20, 2019

Currently, KeyedHashDB trait is used only in tests. To make it test-only, it may be a bit more complicated than just putting #[cfg(test)] on top of it, see #10314 (comment).
@ordian ordian added F6-refactor 📚 Code needs refactoring. P7-nicetohave 🐕 Issue is worth doing eventually. labels Feb 20, 2019
@ordian
Copy link
Collaborator Author

ordian commented Sep 24, 2019

No longer relevant after #10929.

@ordian ordian closed this as completed Sep 24, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
F6-refactor 📚 Code needs refactoring. P7-nicetohave 🐕 Issue is worth doing eventually.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@ordian