Skip to content

Conversation

@FoamyGuy
Copy link
Collaborator

I created a new top level director USB_Host_Examples/ and put these mouse demos inside of it. I am working on the USB Host keyboard guide next and plan to put the keyboard examples in this new folder as well. In the future theoretically some of the older USB host projects could be moved into this folder if we wanted as well, in order to consolidate things outside of the root directory.

@FoamyGuy
Copy link
Collaborator Author

I am not sure why the pylint action failed for this PR. It seems to be indicating warnings in some code that was not modified by this branch.

Run ./pylint_check.sh
Using pylint bin at /opt/hostedtoolcache/Python/3.10.17/x64/bin/pylint
************* Module game
Metro/Metro_RP2350_Chips_Challenge/game.py:81: Too many positional arguments for constructor call (too-many-function-args)

--
Your code has been rated at 10.00/10

@FoamyGuy
Copy link
Collaborator Author

I looked into the line that was specified by the pylint warning and found that it only had 2 positional arguments which matches the class being instantiated, and I don't think is over any limit.

I just re-ran the failed action task and it has succeeded when it ran the second time.

Best guess is there was some kind of corruption on the disk when it ran the first time.

@FoamyGuy FoamyGuy requested a review from TheKitty April 22, 2025 20:14
@TheKitty TheKitty requested a review from tannewt April 24, 2025 18:24
Copy link
Member

@tannewt tannewt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These examples look good to me!

@tannewt tannewt merged commit 37afebb into adafruit:main Apr 24, 2025
51 checks passed
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.

2 participants