Skip to content
This repository was archived by the owner on Jul 27, 2020. It is now read-only.

✨ Cli interface to list connected devices #15

Merged
merged 8 commits into from
Feb 21, 2020
Merged

Conversation

muchzill4
Copy link
Contributor

Description

Presentation layer for our devices listing.

  • Uses tabulate to format output - this can probably be extracted to mbed-tools-lib once we use it in more than one place

Screenshot 2020-02-21 at 11 47 58

Test Coverage

  • This change is covered by existing or additional automated tests.
  • Manual testing has been performed (and evidence provided) as automated testing was not feasible.
  • Additional tests are not required for this change (e.g. documentation update).

@codecov
Copy link

codecov bot commented Feb 21, 2020

Codecov Report

Merging #15 into master will increase coverage by 4.73%.
The diff coverage is 95.23%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #15      +/-   ##
==========================================
+ Coverage   86.46%   91.19%   +4.73%     
==========================================
  Files          14       15       +1     
  Lines         266      284      +18     
==========================================
+ Hits          230      259      +29     
+ Misses         36       25      -11
Impacted Files Coverage Δ
mbed_devices/device.py 100% <100%> (+100%) ⬆️
mbed_devices/mbed_tools/cli.py 100% <100%> (ø)
mbed_devices/mbed_devices.py 75% <75%> (+75%) ⬆️

@muchzill4 muchzill4 requested a review from a team February 21, 2020 13:47
@muchzill4 muchzill4 merged commit acee2eb into master Feb 21, 2020
@muchzill4 muchzill4 deleted the listing-cli branch February 21, 2020 15:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants