Skip to content

Conversation

umbynos
Copy link
Contributor

@umbynos umbynos commented Sep 27, 2021

I have added a new sub-command to be able to retrieve the firmware version in use by a module.
During the process, I refactored a bit the code, since there were useless repetitions.
Docs have been added

@umbynos umbynos added type: enhancement Proposed improvement topic: documentation Related to documentation for the project topic: code Related to content of the project itself labels Sep 27, 2021
@umbynos umbynos requested a review from a team September 27, 2021 16:08
@umbynos umbynos self-assigned this Sep 27, 2021
@umbynos umbynos changed the title add ´get-version´ command add get-version command Sep 28, 2021
Copy link

@ubidefeo ubidefeo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested: works as expected

Co-authored-by: per1234 <accounts@perglass.com>
@codecov-commenter
Copy link

codecov-commenter commented Sep 30, 2021

Codecov Report

Merging #116 (00140b2) into main (59157c1) will decrease coverage by 0.45%.
The diff coverage is 2.65%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #116      +/-   ##
==========================================
- Coverage   11.49%   11.04%   -0.46%     
==========================================
  Files          18       21       +3     
  Lines        1722     1793      +71     
==========================================
  Hits          198      198              
- Misses       1466     1537      +71     
  Partials       58       58              
Flag Coverage Δ
unit 11.04% <2.65%> (-0.46%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
cli/arguments/arguments.go 0.00% <0.00%> (ø)
cli/certificates/flash.go 0.00% <0.00%> (ø)
cli/common/common.go 0.00% <0.00%> (ø)
cli/firmware/firmware.go 0.00% <0.00%> (ø)
cli/firmware/flash.go 0.00% <0.00%> (ø)
cli/firmware/getversion.go 0.00% <0.00%> (ø)
flasher/flasher.go 0.00% <0.00%> (ø)
flasher/nina.go 0.00% <0.00%> (ø)
flasher/winc.go 0.00% <0.00%> (ø)
indexes/firmwareindex/firmwareindex.go 56.71% <ø> (ø)
... and 4 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 66a28f5...00140b2. Read the comment docs.

@umbynos umbynos merged commit 9220c33 into main Sep 30, 2021
@umbynos umbynos deleted the umbynos/get-version branch September 30, 2021 09:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: code Related to content of the project itself topic: documentation Related to documentation for the project type: enhancement Proposed improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants