Skip to content

Refactor bestool to be more managable.#12

Draft
shymega wants to merge 16 commits intoRalim:mainfrom
shymega:refactor/shymega
Draft

Refactor bestool to be more managable.#12
shymega wants to merge 16 commits intoRalim:mainfrom
shymega:refactor/shymega

Conversation

@shymega
Copy link
Contributor

@shymega shymega commented Feb 17, 2024

WIP branch. DRAFT only.

@shymega shymega force-pushed the refactor/shymega branch 9 times, most recently from fe9dd04 to 27011eb Compare February 18, 2024 18:21
@shymega shymega force-pushed the refactor/shymega branch 8 times, most recently from f3c1ccd to 23d2592 Compare March 3, 2024 23:24
@shymega shymega force-pushed the refactor/shymega branch 5 times, most recently from 6d9a9bc to cac6185 Compare March 16, 2024 15:00
@shymega shymega force-pushed the refactor/shymega branch from c9504f9 to 06b10f7 Compare May 9, 2024 15:25
shymega added 7 commits July 19, 2024 20:07
This is part of the refactor.

Signed-off-by: Dom Rodriguez <shymega@shymega.org.uk>
- Use Cargo 'workspaces', specify shared dependencies.
- Move blobs to `src/beslib/src/blobs`, so that Nix can build in a
  sandbox.
- Copy `supply-chain` directory to both `beslib` and `bestool`. TODO:
  Regenerate once refactor complete.

Signed-off-by: Dom Rodriguez <shymega@shymega.org.uk>
Signed-off-by: Dom Rodriguez <shymega@shymega.org.uk>
Signed-off-by: Dom Rodriguez <shymega@shymega.org.uk>
Signed-off-by: Dom Rodriguez <shymega@shymega.org.uk>
Signed-off-by: Dom Rodriguez <shymega@shymega.org.uk>
Signed-off-by: Dom Rodriguez <shymega@shymega.org.uk>
shymega added 9 commits July 19, 2024 20:07
Signed-off-by: Dom Rodriguez <shymega@shymega.org.uk>
Signed-off-by: Dom Rodriguez <shymega@shymega.org.uk>
Discussed with @Ralim over Telegram, and initial feedback was to use
`Write` trait, as for a transport-agnostic writer.

However, I'm not sure how this will fit in with `serialport`.
@shymega shymega force-pushed the refactor/shymega branch from 06b10f7 to fca9dcd Compare July 19, 2024 19:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant