-
Notifications
You must be signed in to change notification settings - Fork 50
waveshare2in13v3/waveshare2in13v4: adding v3 and v4 waveshare 2.13" ePaper displays #65
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
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #65 +/- ##
=======================================
+ Coverage 49.5% 51.1% +1.6%
=======================================
Files 90 99 +9
Lines 11736 12728 +992
=======================================
+ Hits 5808 6500 +692
- Misses 5674 5958 +284
- Partials 254 270 +16 ☔ View full report in Codecov by Sentry. |
|
v4 works great (v3 not tested, no v3 display...)! thanks! |
|
I forgot to comment here, can you fix the linter errors? You'll need to rebase. |
|
rerunning |
Remove unreferenced, unexported values.
Remove unreferenced, unexported constants.
|
@maruel I've fixed the outstanding lint issues on the. Are you OK with me merging it? |
maruel
left a comment
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! It's really appreciated.
…Paper displays (periph#65) * adding v3 and v4 waveshare * updating file * Update waveshare213v4.go Remove unreferenced, unexported values. * Update waveshare213v3.go Remove unreferenced, unexported constants. --------- Co-authored-by: gsexton <georges@mhsoftware.com>
doesn't have a specific issue.
new functionality (if relevant).