Skip to content
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 library, example and docs #1

Merged
merged 9 commits into from
Dec 6, 2023
Merged

Conversation

BlitzCityDIY
Copy link
Contributor

Adding library, example and docs for the FT5336 touch driver

@BlitzCityDIY
Copy link
Contributor Author

hihi @makermelissa - since you've worked on touch drivers before would you mind reviewing this?

Copy link
Contributor

@makermelissa makermelissa left a comment

Choose a reason for hiding this comment

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

Looks pretty good. Just one small change request. Also, nice use of the register library. I may have to update the CSTXXX library in the same way.

docs/conf.py Outdated
Comment on lines 29 to 30
"adafruit_bus_device",
"adafruit_register",
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't think these 2 lines are necessary here if they are libraries. Especially since you already have them under requirements.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

thanks @makermelissa - i tried removing them but it caused CI to fail so i think they are needed

Copy link
Contributor

Choose a reason for hiding this comment

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

Hmm, the error is odd and that adding it there fixed it. Neither of the 2 previous touch libs needed an autodoc_mock_imports and it's really only used for adding internal modules that haven't been added to Blinka.

I'll see if I can find more info on the error.

Copy link
Contributor

Choose a reason for hiding this comment

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

Can you try removing all of the autodoc_mock_imports? Really, none of them should be needed.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ah, that did it. thank you!

Copy link
Contributor

Choose a reason for hiding this comment

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

Yw and thank you. :)

@BlitzCityDIY BlitzCityDIY merged commit c783533 into adafruit:main Dec 6, 2023
1 check 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