Skip to content
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

Use portable atomic, remove explicit/redundant feature declaration & move to edition 2021 #103

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

ithinuel
Copy link
Collaborator

No description provided.

@ithinuel ithinuel force-pushed the use-portable-atomic branch 3 times, most recently from 75046d1 to 068dae0 Compare November 26, 2023 23:44
@ithinuel ithinuel force-pushed the use-portable-atomic branch from 6b5ac37 to 54dfe17 Compare December 6, 2023 00:36
@mfp20
Copy link

mfp20 commented Aug 23, 2024

Please merge this pull request to enable use for AVR and other archs. It externalizes the management of missing atomics to specialized crate (portable-atomic).

@Sympatron
Copy link
Collaborator

I would keep the defmt_0_3 feature, because defmt v1 will be released soon and can be supported simultaneously that way.

@jamesmunns: Is bbqueue still maintained or should it be forked for future development?

@jamesmunns
Copy link
Owner

@Sympatron I'm not actively maintaining it at the moment, I've added @ithinuel as a maintainer, but I think they are busy as well.

If you'd like to be added as a maintainer, I'd be happy to add you.

@Sympatron
Copy link
Collaborator

That would be nice. I would like to help maintain bbqueue if you two are short on time.

@jamesmunns
Copy link
Owner

@Sympatron sent you an invite! Feel free to merge PRs as you see fit. Feel free to ping me if you want to coordinate a release.

@Sympatron
Copy link
Collaborator

@ithinuel Are you ok with me making changes to this PR to resolve conflicts? And as I said, I would like to keep the defmt_0_3 feature. Or do you want to do it yourself?

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.

4 participants