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

Ship a 1.0 release of atsamd-hal #334

Open
jessebraham opened this issue Nov 24, 2020 · 4 comments
Open

Ship a 1.0 release of atsamd-hal #334

jessebraham opened this issue Nov 24, 2020 · 4 comments
Labels
help wanted Extra attention is needed

Comments

@jessebraham
Copy link
Contributor

jessebraham commented Nov 24, 2020

This is still a work in progress.

Milestone:
https://github.com/atsamd-rs/atsamd/milestone/1

Currently blocked by:

If you feel any additional issues should be addressed please comment below or let us know in Matrix.

@jessebraham jessebraham added the help wanted Extra attention is needed label Nov 24, 2020
@jessebraham jessebraham pinned this issue Nov 24, 2020
@bradleyharden
Copy link
Contributor

What's the position on SemVer and incrementing the major version?

I ask, because 1.0 implies API stability. But, as mentioned a few times, many of the APIs are limited to a small subset of the given peripheral's functionality. I'm not surprised or upset by that. Implementing a complete driver is a lot of work. But if someone wants to rewrite a module to be more configurable, that might need to break the existing API.

In that case, would we take the v1/v2 approach, like the gpio module? Then we would drop support for v1 in the next major revision? Or would we just increment the major version immediately and not worry about maintaining support for the old API?

@TDHolmes
Copy link
Contributor

TDHolmes commented Feb 13, 2021 via email

@Sympatron
Copy link
Contributor

I think the API is currently not stable enough. I suppose we want to update all modules to use gpio v2 eventually and I would personally not go 1.0 until that is done. I would even deprecate v1 (or even remove it) and set v2 as the default before switching to 1.0. This shouldn't be rushed IMO, because it doesn't need to. Once you declare the API stable with 1.0 you have to be much more careful when you want to change/improve things.

@jbeaurivage jbeaurivage mentioned this issue Jun 1, 2021
8 tasks
@jacobrosenthal
Copy link
Member

I think #426 should be in here for 1.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

5 participants