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

enable unittests #81

Merged
merged 5 commits into from
Mar 15, 2024
Merged

enable unittests #81

merged 5 commits into from
Mar 15, 2024

Conversation

nimrof
Copy link
Collaborator

@nimrof nimrof commented Mar 12, 2024

Unittests work now, but the tests fail.
So need to fix the tests before merge

@nimrof nimrof marked this pull request as draft March 12, 2024 18:53
@nimrof nimrof self-assigned this Mar 12, 2024
@nimrof nimrof added the enhancement New feature or request label Mar 12, 2024
@nimrof nimrof mentioned this pull request Mar 14, 2024
@nimrof nimrof marked this pull request as ready for review March 15, 2024 12:54
@nimrof nimrof requested a review from trojanobelix March 15, 2024 12:55
@nimrof
Copy link
Collaborator Author

nimrof commented Mar 15, 2024

@trojanobelix @trojanobelix Open for review, just do not merge this one until after #82 is merged and all checks are hopefully green :)

@nimrof nimrof merged commit 52384a5 into CANopenNode:main Mar 15, 2024
1 check passed
@nimrof nimrof deleted the enable_unittests branch March 15, 2024 14:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants