Skip to content

Fix stray slash#411

Merged
FoamyGuy merged 2 commits intoadafruit:mainfrom
tekktrik:fix/stray-slash
Mar 23, 2026
Merged

Fix stray slash#411
FoamyGuy merged 2 commits intoadafruit:mainfrom
tekktrik:fix/stray-slash

Conversation

@tekktrik
Copy link
Member

I had a CI (pylint, I believe?) run warn about this, and after checking how this renders I believe it is just a stray slash. Removing it doesn't change the inteded rendering on the GitHub page:

https://github.com/adafruit/Adafruit_CircuitPython_Bundle/blob/main/circuitpython_library_list.md

@tekktrik tekktrik requested a review from a team March 20, 2026 02:17
@tekktrik tekktrik requested a review from FoamyGuy March 20, 2026 20:33
Copy link
Contributor

@FoamyGuy FoamyGuy left a comment

Choose a reason for hiding this comment

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

Thank you!

@FoamyGuy FoamyGuy merged commit 8d94e16 into adafruit:main Mar 23, 2026
3 checks passed
@tekktrik
Copy link
Member Author

tekktrik commented Mar 23, 2026

@FoamyGuy - Looking at the code now, this will probably cause adabot to do some funky things to the library list like duplicate the entries or report that changes were made because it will inadvertently detect a new library was added. I'll update the lists in both bundles today to try to get ahead of the next run at roughly 6:30 UTC (or midnight in the US).

@tekktrik
Copy link
Member Author

Also, the CI failed for merging this, but I think it's a permissions error if I had to guess (with respect to API or REST calls)?

@FoamyGuy
Copy link
Contributor

I looked into the actions failure this morning. I think its possible the token in use may be expired.

The tests run successfully locally with personal token.

@tannewt
Copy link
Member

tannewt commented Mar 23, 2026

I fixed the CI by updating the personal access token used for Blinka. (It is no-perm, just read public repos and working ok.)

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.

3 participants