search

jgillula / rpi-rfm69

Python RFM69 library for Raspberry Pi
GNU General Public License v3.0
31 stars 20 forks source link

Update hookup.rst #4

Closed yheynema closed 3 years ago

yheynema commented 5 years ago

Suggesting a doc fix in PI Pin. The PI Pin for ID_SC shows 18... I think the correct one should be 28... typo? Also, in the code, under radio.py (self.intPin = kwargs.get('interruptPin', 18)) is that right?

jgillula commented 3 years ago

Hi there! I recently look over as maintainer for this repo, and I'm trying to close out some old PRs.

I think you're right about the documentation, but I hesitate to change the defaults for the keyword arguments in case someone out there is relying on them. I'll go ahead and merge your commit (and line it up for release in the next version) since it only changes the documentation.

Thanks for catching this!