-
Notifications
You must be signed in to change notification settings - Fork 11
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 Append command #53
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for the PR. It looks useful.
One required change as per my suggestion.
Could you please add one-or-more tests? Similar, easily adaptable ones are for example test_connect_put_multi_line()
in tests/test_pandablocks.py
. Any of the test_connect_put_*
tests should show you examples.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the contribution! I've made a couple of suggestions to simplify. I'll second Alex's request for a test if you have a chance...
Co-authored-by: Alexander Wells <[email protected]>
Co-authored-by: Tom C (DLS) <[email protected]>
Co-authored-by: Tom C (DLS) <[email protected]>
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #53 +/- ##
==========================================
+ Coverage 96.35% 96.37% +0.02%
==========================================
Files 12 12
Lines 1151 1159 +8
==========================================
+ Hits 1109 1117 +8
Misses 42 42
☔ View full report in Codecov by Sentry. |
Add Append command for SEQ table usage.