stackabletech / stackabletech/operator-rs
Add type state builders
Open
@Techassi is already working on this.
Since Jun 15, 2023.
status/blocked
- Dominant language
- Rust
- Stars
- 167
- Forks
- 19
- Avg merge
- 1d 6h
- Merged PRs (30d)
- 9
Description
This issue tracks the effort to implement the type state pattern for all builders across operator-rs. This involves updating multiple builders, adjusting tests and updating all product operators.
Additionally, an upstream PR in typed_builder should make it suitable to implement the pattern automatically with derive macros.
### Tasks
- [ ] https://github.com/idanarye/rust-typed-builder/pull/94
- [ ] Evaluate typed_builder for our usecases
- [ ] Implement pattern across all builders. This includes method renaming
- [ ] https://github.com/idanarye/rust-typed-builder/pull/95
- [ ] https://github.com/stackabletech/operator-rs/pull/615
Contributor guide
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.