Skip to content

remove dotstar_featherwing #248

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

Closed
wants to merge 1 commit into from

Conversation

FoamyGuy
Copy link
Contributor

It appears this repo was deleted https://github.com/jepler/circuitPython_dotstar_featherwing.git. this causes git submodule update to fail.

Removing the submodule to it resolves that issue.

It seems like the functionality may have ended up in an adafruit library here: https://github.com/adafruit/Adafruit_CircuitPython_FeatherWing/blob/main/adafruit_featherwing/dotstar_featherwing.py

@FoamyGuy
Copy link
Contributor Author

@jepler was the circuitpython_dotstar_featherwing repo deleted intentionally? If so, do you know if the helper class in adafruit_featherwing covers the same functionality?

@jepler
Copy link
Contributor

jepler commented Apr 28, 2025

uh oh. I was deleting forks of Adafruit stuff within my own github account and that included this one.

#216 has the reasons for this change.

I will re-fork the repo into my github user and put the main branch back where it was. and try not to delete it this time.

@jepler
Copy link
Contributor

jepler commented Apr 28, 2025

I would be happy to transfer this repo into the adafruit or circuitpython organization, as I don't intend to maintain it.

@FoamyGuy
Copy link
Contributor Author

I think moving it to circuitpython org sounds good, I'm in the process of moving the libraries there into the community bundle anyhow so it will have company there.

@jepler
Copy link
Contributor

jepler commented Apr 28, 2025

I've requested to transfer the repo to your personal account @FoamyGuy and then you can transfer it into the correct organization. sorry for the trouble!

@FoamyGuy
Copy link
Contributor Author

Sounds good, Thank you! I'll get it moved.

Closing this PR, I'll submit a seperate PR that changes the submodule to it's new repo once that is up.

@FoamyGuy FoamyGuy closed this Apr 28, 2025
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