Consider changing enums to struct(u16)
Open
@jcjones is already working on this.
Since Jul 8, 2026.
- Dominant language
- Rust
- Stars
- 70
- Forks
- 16
- Avg merge
- 4h 36m
- Merged PRs (30d)
- 30
Description
J.C.'s thought: we use enum all over, but I'm tempted to make this
#[derive(Clone, Copy, Debug, PartialEq, Eq, PartialOrd, Ord, Hash)]
pub struct CollectionJobExtensionType(u16);
so the boilerplate becomes more compiler-generated.
Originally posted by @jcjones in https://github.com/divviup/janus/pull/4693#discussion_r3540220103
Give an example of how it goes. Consider documenting the preference either way in the DEVELOPMENT.md
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.