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

Make IO port constant names consistent #3

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Sanqui
Copy link

@Sanqui Sanqui commented Apr 3, 2024

Previously, some pins were defined with a single digit and some with a leading zero. Some pins were defined with _OUT and some without. This commit adds aliases for all variants.

I know this is a simple pull request, but I just lost an hour trying to debug my setup until I found out my config has to look like this:

set DRIVE_5_SELECT_LINE DRIVES_PORT_PIN13_OUT
set DRIVE_5_MOTOR_LINE  DRIVES_PORT_PIN6

So adding aliases for all variants should prevent some future headaches. :)

Previously, some pins were defined with a single digit and some
with a leading zero.  Some pins were defined with _OUT and some
without.  This commit adds aliases for all variants.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant