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

Remove or enable the async feature #2007

Closed
MabezDev opened this issue Aug 27, 2024 · 0 comments · Fixed by #2070
Closed

Remove or enable the async feature #2007

MabezDev opened this issue Aug 27, 2024 · 0 comments · Fixed by #2070
Assignees
Labels
RFC Request for comment
Milestone

Comments

@MabezDev
Copy link
Member

MabezDev commented Aug 27, 2024

By default. We previously had this as a feature because of the way we were doing interrupts meant that there were linker conflicts. Now that we do runtime binding of interrupts, we could enable this by default, or ditch the feature entirely. The only argument I can think of for keeping it is for compile time reasons, but I can't see it making a big dent overall.

@MabezDev MabezDev added this to the 0.21.0 milestone Aug 27, 2024
@MabezDev MabezDev added the RFC Request for comment label Aug 30, 2024
@jessebraham jessebraham self-assigned this Aug 30, 2024
@github-project-automation github-project-automation bot moved this from Todo to Done in esp-rs Sep 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
RFC Request for comment
Projects
Archived in project
2 participants