-
Notifications
You must be signed in to change notification settings - Fork 193
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
Adding VCNL4020 #442
Adding VCNL4020 #442
Conversation
Adding VCNL4020 library to the bundle under drivers.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I only see 1 small issue, otherwise it looks good.
.gitmodules
Outdated
[submodule "libraries/drivers/vcnl4020"] | ||
path = libraries/drivers/vcnl4020 | ||
url = https://github.com/adafruit/Adafruit_CircuitPython_VCNL4020.git | ||
[submodule "libraries/drivers/libraries/drivers/vcnl4020"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks like you accidentally left a duplicate submodule.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, looks good.
I can log in as adabot on RTD, but don't see any invites. Maybe they go to some mailbox? |
@makermelissa hrm i'm not sure unfortunately, it just shows pending on my end. @tannewt - do the RTD adabot invites go to an email mailbox? |
@BlitzCityDIY @makermelissa answering re RTD mails internally. |
Adding VCNL4020 library to the bundle under drivers. If someone with access to ReadTheDocs is able to accept the adabot invite and add to CircuitPython as a subproject I'd appreciate it