Skip to content

Move cap1188 to drivers folder. #98

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 3 commits into from
Closed

Move cap1188 to drivers folder. #98

wants to merge 3 commits into from

Conversation

caternuson
Copy link
Contributor

Accidentally put the CAP1188 driver one folder up. This PR moves it to the correct location and updates .gitmodules.

Copy link
Contributor

@kattni kattni left a comment

Choose a reason for hiding this comment

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

It looks like this PR is trying to add in all the submodule files as well as your change. The only thing that should be changing is .gitmodules, libraries/drivers/cap1188 and /libraries/cap1188. Please look into what's going on here. Ping me if you need help with it.

@caternuson
Copy link
Contributor Author

Not sure how that happened. Manually cleaned up. Hopefully this is OK now.

Copy link
Contributor

@kattni kattni left a comment

Choose a reason for hiding this comment

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

You still need to add the submodule in the same way you did before, there should be something like submodule cap1188 added at <hash> included in changed files.

.gitmodules Outdated
[submodule "libraries/cap1188"]
path = libraries/cap1188
[submodule "libraries/drivers/cap1188"]
path = libraries/drivers/cap1188
url = https://github.com/adafruit/Adafruit_CircuitPython_Bundle.git
Copy link
Contributor

Choose a reason for hiding this comment

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

I believe the URL here is incorrect.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

it totally is

@caternuson
Copy link
Contributor Author

This PR is just all kinds of messed up. Sending it to the dungeon.

@caternuson caternuson closed this Oct 12, 2018
@kattni
Copy link
Contributor

kattni commented Oct 12, 2018

Closed in favor of #99

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