Ron Evans
Ron Evans
@racerxdl can you please rebase this branch against `dev` to resolve the merge conflicts? Thank you.
Hello, everyone. I would love to help get this PR merged before the next release. Seems like there are again (or still?) some merge conflicts to resolve. After that, what...
> If possible, please add a smoke test for this PR. It will make it obvious when there is a compile-time breaking change. Yes, please, @racerxdl is this possible? Ideally...
Thanks @racerxdl that sounds great!
The fix for #2348 has been merged into `dev` branch, just to let you know @sago35
Sorry that I am only now finally looking at this more closely. Maybe it should be called `PWMforPin()` that way it can be implemented for all PWM interfaces, including ones...
`func PWMforPin(Pin) (pwmer,error)` as long as the interface is the same seems like a good approach to me.
Any opinions on this PR @aykevl @conejoninja @sago35 anyone? :smile_cat:
This change is not intended to remove dependency on `machine`. It is only to simplify the syntax for default values. One benefit is it removes the need to explain "what...
> Although, maybe the suggestion from @conejoninja is better as it also allows using the pins in an interface. That right there does achieve both the simplification I am seeking,...