-
Notifications
You must be signed in to change notification settings - Fork 14
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
Add D1 platform (2023 edition) #69
Conversation
✅ Deploy Preview for merry-scone-cc7a60 ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
nice |
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.
you love to see it!!
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.
Some nits, but this is awesome!
@hawkw breaking this out as a longer response:
So, some of these things COULD be portable to any user of the D1, however there's a very fuzzy line today. I haven't really tackled any of the "portability challenges", like handling different configs, or pin mapping, or what DMA channel to use, or even how to handle interrupts. These absolutely should live in the |
Great, thanks — I was mostly just curious about whether this could ever move to the |
No description provided.