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

Add MCP4725 DAC Component #1418

Merged
merged 8 commits into from
Feb 6, 2021
Merged

Conversation

JJK801
Copy link
Contributor

@JJK801 JJK801 commented Dec 15, 2020

Description:

Related issue (if applicable): fixes esphome/feature-requests#1031

Pull request in esphome-docs with documentation (if applicable): esphome/esphome-docs#889

Checklist:

  • The code change is tested and works locally.
  • Tests have been added to verify that the new code works (under tests/ folder).

If user exposed functionality or configuration variables are added/changed:

@glmnet
Copy link
Member

glmnet commented Dec 22, 2020

GTM when codeowners and test added, docs too, thanks!

@glmnet glmnet merged commit 04d8593 into esphome:dev Feb 6, 2021
@glmnet
Copy link
Member

glmnet commented Feb 6, 2021

Thanks!!

@JJK801
Copy link
Contributor Author

JJK801 commented Feb 6, 2021

Thanks to you @glmnet, i planned to add tests but i was too busy last month 👍

@JJK801 JJK801 deleted the feat/add-mcp4725-i2c-dac branch February 6, 2021 15:22
@alex-shaw
Copy link

Wanted to leave my thanks also - Im also keen to use this peripheral!

taquitos pushed a commit to taquitos/esphome that referenced this pull request Mar 14, 2021
* Add MCP4725 DAC

* Fix lint

* Fix lint

* Fix lint

* Lint & cleanup

* Lint again

* One more lint

* add test

Co-authored-by: Guillermo Ruffino <glm.net@gmail.com>
This was referenced Mar 22, 2021
@github-actions github-actions bot locked and limited conversation to collaborators Sep 14, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support for mcp4725 - 3.3v 12 bit i2c dac with EEPROM
3 participants