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.

Sign up for GitHub

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

Jump to bottom

Bug/6 close serial #12

Merged
merged 2 commits into from
Jan 16, 2018
Merged

Bug/6 close serial #12

merged 2 commits into from
Jan 16, 2018

Conversation

TJonesy
Copy link
Contributor

@TJonesy TJonesy commented Jan 16, 2018

Add destructor to close port on leaving scope and relevant tests.

#6

@TJonesy TJonesy added the bug label Jan 16, 2018
@TJonesy TJonesy requested a review from konnorandrews January 16, 2018 00:38
@coveralls
Copy link

coveralls commented Jan 16, 2018 •
edited
Loading

Coverage Status

Coverage increased (+0.03%) to 97.015% when pulling 753e52a on bug/6_close_serial into 71ae4ed on develop.

@konnorandrews konnorandrews merged commit 0ecdc43 into develop Jan 16, 2018
@konnorandrews konnorandrews deleted the bug/6_close_serial branch January 16, 2018 00:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@konnorandrews konnorandrews konnorandrews approved these changes

Assignees

@konnorandrews konnorandrews

Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@TJonesy @coveralls @konnorandrews