feat: Add host CLI option as alternative to config#1395
feat: Add host CLI option as alternative to config#1395
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1395 +/- ##
==========================================
+ Coverage 95.30% 95.34% +0.03%
==========================================
Files 43 43
Lines 3198 3201 +3
==========================================
+ Hits 3048 3052 +4
+ Misses 150 149 -1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
c3e9908 to
136df8d
Compare
3fde52e to
2d156a3
Compare
2cbe490 to
2533134
Compare
2533134 to
67ee7cb
Compare
67ee7cb to
3d17702
Compare
abbiemery
left a comment
There was a problem hiding this comment.
When trying to run scans against tiled 0.2.4 (current state of main) with this I am getting an error from the tiled writer:
TypeError: DataSource.__init__() got an unexpected keyword argument 'properties'
This may be just down to it needing rebasing but does raise the question of if we are clear somewhere about the versions of tiled we support.
Works against 0.2.3. Can login/out, get devices and run plans etc. Happy to approve pending a credential discussion.
|
Keep the new CLI option to set the host but remove the stomp config stuff and replace with websocket work |
8d41f65 to
b0f3611
Compare
b0f3611 to
a17edfd
Compare
a17edfd to
54fe23b
Compare
|
@tpoliaw So as it stands you can look at devices etc with the host option, but can't run plans? |
54fe23b to
5936ed3
Compare
|
You can run plans in the background but nothing that requires stomp |
|
Have added docs to host command and made |

No description provided.