Skip to content
Merged
Changes from 1 commit
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
a33072e
added more mapper in feedback result pkg
alessio-perugini Oct 25, 2023
e639e5b
cli: outdated command use feedback result structs
alessio-perugini Oct 25, 2023
cbe3b0d
cli: upload use feedback result structs
alessio-perugini Oct 25, 2023
3ae9dee
cli: board details use feedback result structs
alessio-perugini Oct 25, 2023
da7541a
rename result Platform in PlatformSummary
alessio-perugini Oct 25, 2023
3aee471
fixup mapper BoardListAllRespnse
alessio-perugini Oct 25, 2023
dcf7f86
cli: board listall use feedback result structs
alessio-perugini Oct 25, 2023
cdc80bf
cli: board search use feedback result structs
alessio-perugini Oct 25, 2023
60abc6a
cli: board list use feedback result structs
alessio-perugini Oct 25, 2023
1534ab9
cli: lib deps use feedback result structs
alessio-perugini Oct 25, 2023
5fd4539
cli: lib search use feedback result structs
alessio-perugini Oct 25, 2023
c93f4bc
cli: lib examples use feedback result structs
alessio-perugini Oct 25, 2023
e18ec5f
cli: lib list use feedback result structs
alessio-perugini Oct 25, 2023
ce31a9b
cli: monitor use feedback result structs
alessio-perugini Oct 25, 2023
429f525
cli: compile use feedback result structs
alessio-perugini Oct 25, 2023
a16ba4d
add tests
alessio-perugini Oct 26, 2023
e619093
fix typo and import name
alessio-perugini Nov 6, 2023
2e16184
remove redundant nil checks
alessio-perugini Nov 6, 2023
0702b77
fix printing empty string instead of empty table
alessio-perugini Nov 6, 2023
90cb43a
fix: make anonymous struct explicit
alessio-perugini Nov 6, 2023
2d77b39
fix: use rpc enum String method
alessio-perugini Nov 6, 2023
a126e93
add BoardListWatchResponse mapper
alessio-perugini Nov 6, 2023
0aabc1b
fix: remap rpc enums
alessio-perugini Nov 6, 2023
1d855a0
fix: cli sketch new - not showing any json output
alessio-perugini Nov 6, 2023
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
cli: board listall use feedback result structs
  • Loading branch information
alessio-perugini committed Oct 26, 2023
commit dcf7f86ddea7a5724b500295b00cd6019f90eea6
23 changes: 16 additions & 7 deletions internal/cli/board/listall.go
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@ import (

"github.com/arduino/arduino-cli/commands/board"
"github.com/arduino/arduino-cli/internal/cli/feedback"
fResult "github.com/arduino/arduino-cli/internal/cli/feedback/result"
"github.com/arduino/arduino-cli/internal/cli/instance"
rpc "github.com/arduino/arduino-cli/rpc/cc/arduino/cli/commands/v1"
"github.com/arduino/arduino-cli/table"
Expand Down Expand Up @@ -63,32 +64,40 @@ func runListAllCommand(cmd *cobra.Command, args []string) {
feedback.Fatal(tr("Error listing boards: %v", err), feedback.ErrGeneric)
}

feedback.PrintResult(resultAll{list})
feedback.PrintResult(resultAll{fResult.NewBoardListAllResponse(list)})
}

// output from this command requires special formatting, let's create a dedicated
// feedback.Result implementation
type resultAll struct {
list *rpc.BoardListAllResponse
list *fResult.BoardListAllResponse
}

func (dr resultAll) Data() interface{} {
return dr.list
}

func (dr resultAll) String() string {
t := table.New()
t.SetHeader(tr("Board Name"), tr("FQBN"), "")

if dr.list == nil || len(dr.list.Boards) == 0 {
return t.Render()
}

sort.Slice(dr.list.Boards, func(i, j int) bool {
return dr.list.Boards[i].GetName() < dr.list.Boards[j].GetName()
return dr.list.Boards[i].Name < dr.list.Boards[j].Name
})

t := table.New()
t.SetHeader(tr("Board Name"), tr("FQBN"), "")
for _, item := range dr.list.GetBoards() {
for _, item := range dr.list.Boards {
if item == nil {
continue
}
hidden := ""
if item.IsHidden {
hidden = tr("(hidden)")
}
t.AddRow(item.GetName(), item.GetFqbn(), hidden)
t.AddRow(item.Name, item.Fqbn, hidden)
}
return t.Render()
}