-
Notifications
You must be signed in to change notification settings - Fork 215
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.
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
hal: add optional defmt support. #450
Conversation
Defmt is known by its unstability. Is it good idea to depends on it? |
fair point, i think it is still useful to have / we'll just have to keep it up to date |
How about naming the feature |
Related issue: #460 Making the enums (optionally) derive |
|
04f1f5c
to
d749ef0
Compare
the GHA `include` field is nonsense.
We have 16 jobs now, starts to hit the GHA free limit of 20. There's no advantage in testing thumbv6 and thumbv7 because they're almost identical and nothing in e-h depends on them, so remove thumbv6.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the expected amount of cursed i think but lgtm!
(if we do end up with more defmt versions / features maybe we can split the selection and enabling flags but it's pretty moot for now)
No description provided.