Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

drivers: Add MAX6675 thermocouple digitizer #435

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

Merged
eldruin merged 2 commits into rust-embedded:master from onkoe:master
Dec 20, 2023

Conversation

@onkoe
Copy link
Contributor

@onkoe onkoe commented Dec 20, 2023

This commit adds my new MAX6675 thermocouple digitizer driver to the list. You can find it on GitHub here: https://github.com/onkoe/max6675-hal

If you think any changes are necessary, please let me know! 🤩

This commit adds my new MAX6675 thermocouple digitizer driver to the
list. You can find it on GitHub below:
https://github.com/onkoe/max6675-hal 
I have a Grammarly extension in my IDE, making most things here
errors. However, it caught some hard-to-find issues that
look a lot better fixed!
Please feel free to refuse this commit if you're uninterested in
the changes. ☺️
Copy link
Contributor Author

onkoe commented Dec 20, 2023

Uhh... I'm also going to tack on a commit where I fixed various grammar mistakes in the document.

Feel free to eschew it entirely - I didn't want to spring two random PRs simultaneously!

Copy link
Member

@eldruin eldruin left a comment

Choose a reason for hiding this comment

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

Nice work, thank you!
And also thank you for the numerous corrections :)

@eldruin eldruin merged commit c689c66 into rust-embedded:master Dec 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@eldruin eldruin eldruin approved these changes

@RandomInsano RandomInsano Awaiting requested review from RandomInsano RandomInsano is a code owner

@berkus berkus Awaiting requested review from berkus berkus is a code owner

@rahul-thakoor rahul-thakoor Awaiting requested review from rahul-thakoor rahul-thakoor is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

AltStyle によって変換されたページ (->オリジナル) /