Remove debug prints (arduino#526)

MatteoPologruto · Jan 7, 2020 · 5f4a92e · 5f4a92e
1 parent d83170e
commit 5f4a92e
Show file tree

Hide file tree

Showing 10 changed files with 0 additions and 16 deletions.
diff --git a/arduino/cores/packageindex/index.go b/arduino/cores/packageindex/index.go
@@ -204,7 +204,6 @@ func LoadIndex(jsonIndexFile *paths.Path) (*Index, error) {
 	if err != nil {
 		return nil, err
 	}
-	//fmt.Println(string(buff))
 	var index Index
 	err = json.Unmarshal(buff, &index)
 	if err != nil {

diff --git a/arduino/cores/packageindex/index_test.go b/arduino/cores/packageindex/index_test.go
@@ -16,7 +16,6 @@
 package packageindex
 
 import (
-	"fmt"
 	"testing"
 
 	"github.com/arduino/go-paths-helper"
@@ -33,7 +32,6 @@ func TestIndexParsing(t *testing.T) {
 		if indexFile.Ext() != ".json" {
 			continue
 		}
-		fmt.Println("Loading:", indexFile)
 		_, err := LoadIndex(indexFile)
 		require.NoError(t, err)
 	}

diff --git a/arduino/cores/packagemanager/package_manager_test.go b/arduino/cores/packagemanager/package_manager_test.go
@@ -214,7 +214,6 @@ func TestBoardOptionsFunctions(t *testing.T) {
 func TestFindToolsRequiredForBoard(t *testing.T) {
 	os.Setenv("ARDUINO_DATA_DIR", dataDir1.String())
 	configuration.Init("")
-	fmt.Println(viper.AllSettings())
 	pm := packagemanager.NewPackageManager(
 		dataDir1,
 		configuration.PackagesDir(),

diff --git a/cli/cli_test.go b/cli/cli_test.go
@@ -212,7 +212,6 @@ func detectLatestAVRCore(t *testing.T) string {
 		}
 	}
 	require.NotEmpty(t, latest, "latest avr core version")
-	fmt.Println("Latest AVR core version:", latest)
 	return latest.String()
 }
 

diff --git a/commands/daemon/daemon.go b/commands/daemon/daemon.go
@@ -19,7 +19,6 @@ package daemon
 
 import (
 	"context"
-	"fmt"
 	"io"
 	"net/http"
 
@@ -114,7 +113,6 @@ func (s *ArduinoCoreServerImpl) Init(req *rpc.InitReq, stream rpc.ArduinoCore_In
 	if err != nil {
 		return err
 	}
-	fmt.Println(resp)
 	return stream.Send(resp)
 }
 

diff --git a/commands/daemon/settings.go b/commands/daemon/settings.go
@@ -19,7 +19,6 @@ import (
 	"context"
 	"encoding/json"
 	"errors"
-	"fmt"
 
 	rpc "github.com/arduino/arduino-cli/rpc/settings"
 	"github.com/spf13/viper"
@@ -62,8 +61,6 @@ func (s *SettingsService) GetValue(ctx context.Context, req *rpc.GetValueRequest
 	key := req.GetKey()
 	value := &rpc.Value{}
 
-	fmt.Println(viper.AllKeys())
-
 	if !viper.InConfig(key) {
 		return nil, errors.New("key not found in settings")
 	}

diff --git a/commands/daemon/settings_test.go b/commands/daemon/settings_test.go
@@ -18,7 +18,6 @@ package daemon
 import (
 	"context"
 	"encoding/json"
-	"fmt"
 	"testing"
 
 	"github.com/spf13/viper"
@@ -54,7 +53,6 @@ func TestMerge(t *testing.T) {
 	_, err := svc.Merge(context.Background(), &rpc.RawData{JsonData: bulkSettings})
 	require.Nil(t, err)
 
-	fmt.Println(viper.AllSettings())
 	require.Equal(t, "420", viper.GetString("daemon.port"))
 	require.Equal(t, "bar", viper.GetString("foo"))
 

diff --git a/commands/instances.go b/commands/instances.go
@@ -148,7 +148,6 @@ func Init(ctx context.Context, req *rpc.InitReq, downloadCB DownloadProgressCB,
 	instances[handle] = instance
 
 	if err := instance.checkForBuiltinTools(downloadCB, taskCB, downloaderHeaders); err != nil {
-		fmt.Println(err)
 		return nil, err
 	}
 

diff --git a/legacy/builder/test/includes_to_include_folders_test.go b/legacy/builder/test/includes_to_include_folders_test.go
@@ -16,7 +16,6 @@
 package test
 
 import (
-	"fmt"
 	"path/filepath"
 	"sort"
 	"testing"
@@ -317,7 +316,6 @@ func TestIncludesToIncludeFoldersSubfolders(t *testing.T) {
 
 	importedLibraries := ctx.ImportedLibraries
 	sort.Sort(ByLibraryName(importedLibraries))
-	fmt.Println(importedLibraries)
 	require.Equal(t, 3, len(importedLibraries))
 	require.Equal(t, "testlib1", importedLibraries[0].Name)
 	require.Equal(t, "testlib2", importedLibraries[1].Name)

diff --git a/legacy/builder/tools_loader.go b/legacy/builder/tools_loader.go
@@ -24,7 +24,6 @@ type ToolsLoader struct{}
 
 func (s *ToolsLoader) Run(ctx *types.Context) error {
 	if ctx.CanUseCachedTools {
-		//fmt.Println("no fs modification, can use cached ctx")
 		return nil
 	}