Skip to content

feat(em511): Change struct of driver - #43

Merged
mirzak merged 2 commits into
mainfrom
new_struct_v2
Oct 31, 2025
Merged

feat(em511): Change struct of driver#43
mirzak merged 2 commits into
mainfrom
new_struct_v2

Conversation

@Dexter9532

@Dexter9532 Dexter9532 commented Oct 30, 2025

Copy link
Copy Markdown
Contributor

Code was the same in many places so did this to make it be cleaner

@Dexter9532
Dexter9532 requested a review from mirzak October 30, 2025 12:33
@Dexter9532
Dexter9532 marked this pull request as ready for review October 30, 2025 12:33

@mirzak mirzak left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Nice, some minor things.

Your commit body message has a typo in it,

Code was the same in many palces so did this to make it be cleaner

Comment thread src/em511/em511.py
Comment thread src/em511/em511.pyi
@Dexter9532
Dexter9532 force-pushed the new_struct_v2 branch 2 times, most recently from 541cd74 to 0a04839 Compare October 30, 2025 13:45
@Dexter9532
Dexter9532 requested a review from mirzak October 30, 2025 13:45
Code was the same in many places so did this to make it be cleaner

Co-authored-by: Mirza Krak <mirza@mkrak.org>
Signed-off-by: Bobo Bäck Engström <bobo@id8-engineering.io>
mirzak
mirzak previously approved these changes Oct 30, 2025
@mirzak
mirzak requested a review from MiladMakdesi October 30, 2025 14:44
Comment thread src/em511/test_em511.py
Comment thread src/em511/test_em511.py

@MiladMakdesi MiladMakdesi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

2 typos, rename tests to appropriate No. order.

Signed-off-by: Mirza Krak <mirza@mkrak.org>
@mirzak

mirzak commented Oct 31, 2025

Copy link
Copy Markdown
Contributor

@MiladMakdesi, you need to approve for us to be able to merge it.

@mirzak
mirzak merged commit a08f876 into main Oct 31, 2025
20 checks passed
@mirzak
mirzak deleted the new_struct_v2 branch October 31, 2025 07:22
This was referenced Oct 31, 2025
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.

3 participants