Skip to content

remove minicbor #52

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
InvertedX merged 30 commits into from
Nov 22, 2025
Merged

remove minicbor #52

InvertedX merged 30 commits into from
Nov 22, 2025

Conversation

@nicoburniske
Copy link
Contributor

@nicoburniske nicoburniske commented Nov 21, 2025 •
edited
Loading

  • remove minicbor
  • implement derive macro (specific to our usage patterns, not a general dcbor macro)
  • remove anyhow in favor thiserror
  • delete obsolete workspace members
  • add a bunch of tests
  • add golden tests

@nicoburniske nicoburniske force-pushed the remove-minicbor branch 2 times, most recently from 0df4a35 to 39d603d Compare November 21, 2025 12:15
@nicoburniske nicoburniske marked this pull request as ready for review November 22, 2025 20:17
@InvertedX InvertedX merged commit e7e8b1b into main Nov 22, 2025
1 check passed
@nicoburniske nicoburniske deleted the remove-minicbor branch November 22, 2025 20:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@nicoburniske @InvertedX