Ron Evans

Results 827 comments of 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,...