Fix linting issues

CONTRIBUTING.md

@@ -1,11 +1,11 @@
 # Blockbook Contributor Guide
 
-Blockbook is back-end service for Trezor wallet. Although it is open source, the design and development of the core packages
+Blockbook is a back-end service for the Trezor wallet. Although it is open source, the design and development of the core packages
-is done by Trezor developers in order to keep Blockbook compatible with Trezor.
+is done by the Trezor developers to keep Blockbook compatible with Trezor.
 
-Bug fixes and support for new coins are welcome. **Please take note that non-fixing pull requests that change base
+Bug fixes and support for new coins are welcome. **Please take note that pull requests that are not fixes and that change base
-packages or another coin code will not be accepted.** If you have a need to change some of the existing code, please file
+packages or another coin code will not be accepted.** If you need a change in the existing core code, please file
-an issue and discuss your request with Blockbook maintainers.
+an issue and discuss your request with the Blockbook maintainers.
 
 ## Development environment
 
@@ -16,8 +16,8 @@ Instructions to set up your development environment and build Blockbook are desc
 
 ### Reporting bugs
 
-A great way to contribute to the project is to send a detailed report when you encounter an issue. We always appreciate
+A great way to contribute to the project is to send a detailed report when you encounter a problem. We always appreciate
-a well-written, thorough bug report, and will thank you for it!
+a well-written and thorough bug report, and we'll be grateful for it!
 
 Check that [our issue database](https://github.com/trezor/blockbook/issues) doesn't already include that problem or
 suggestion before submitting an issue. If you find a match, you can use the "subscribe" button to get notified on

README.md

@@ -5,10 +5,10 @@
 **Blockbook** is back-end service for Trezor wallet. Main features of **Blockbook** are:
 
 - index of addresses and address balances of the connected block chain
-- fast searches in the indexes
+- fast index search
 - simple blockchain explorer
 - websocket, API and legacy Bitcore Insight compatible socket.io interfaces
-- support of multiple coins (Bitcoin and Ethereum type), with easy extensibility for other coins
+- support of multiple coins (Bitcoin and Ethereum type) with easy extensibility to other coins
 - scripts for easy creation of debian packages for backend and blockbook
 
 ## Build and installation instructions
@@ -51,7 +51,8 @@ Please add your experience to this [issue](https://github.com/trezor/blockbook/i
 
 #### Error `internalState: database is in inconsistent state and cannot be used`
 
-Blockbook was killed during the initial import, most commonly by OOM killer. By default, Blockbook performs the initial import in bulk import mode, which for performance reasons does not store all the data immediately to the database. If Blockbook is killed during this phase, the database is left in an inconsistent state. 
+Blockbook was killed during the initial import, most commonly by OOM killer. 
+By default, Blockbook performs the initial import in bulk import mode, which for performance reasons does not store all data immediately to the database. If Blockbook is killed during this phase, the database is left in an inconsistent state. 
 
 See above how to reduce the memory footprint, delete the database files and run the import again. 
 
@@ -67,9 +68,9 @@ Your coin's block/transaction data may not be compatible with `BitcoinParser` `P
 
 #### Cannot build Blockbook using `go build` command
 
-When building Blockbook, I get error `not enough arguments in call to _Cfunc_rocksdb_approximate_sizes`.
+When building Blockbook I get error `not enough arguments in call to _Cfunc_rocksdb_approximate_sizes`.
 
-RocksDB version 6.16.0 changed the API in backward incompatible way. It is necessary to build Blockbook with the tag `rocksdb_6_16`, which fixes the compatibility issue. The correct way to build Blockbook is:
+RocksDB version 6.16.0 changed the API in a backwards incompatible way. It is necessary to build Blockbook with the `rocksdb_6_16` tag to fix the compatibility problem. The correct way to build Blockbook is:
 
 ```
 go build -tags rocksdb_6_16

api/types_test.go

@@ -1,4 +1,4 @@
-// +build unittest
+//go:build unittest
 
 package api
 

bchain/baseparser_test.go

@@ -1,4 +1,4 @@
-// +build unittest
+//go:build unittest
 
 package bchain
 

bchain/coins/bch/bcashparser_test.go

@@ -1,4 +1,4 @@
-// +build unittest
+//go:build unittest
 
 package bch
 

bchain/coins/bellcoin/bellcoinparser_test.go

@@ -1,4 +1,4 @@
-// +build unittest
+//go:build unittest
 
 package bellcoin
 

bchain/coins/bitcore/bitcoreparser_test.go

@@ -1,4 +1,4 @@
-// +build unittest
+//go:build unittest
 
 package bitcore
 

bchain/coins/bitzeny/bitzenyparser_test.go

@@ -1,4 +1,4 @@
-// +build unittest
+//go:build unittest
 
 package bitzeny
 

bchain/coins/btc/bitcoinparser_test.go

@@ -1,5 +1,4 @@
 //go:build unittest
-// +build unittest
 
 package btc
 

bchain/coins/btg/bgoldparser_test.go

@@ -1,4 +1,4 @@
-// +build unittest
+//go:build unittest
 
 package btg
 

bchain/coins/dash/dashparser_test.go

@@ -1,4 +1,4 @@
-// +build unittest
+//go:build unittest
 
 package dash
 

bchain/coins/dcr/decredparser_test.go

@@ -1,5 +1,4 @@
 //go:build unittest
-// +build unittest
 
 package dcr
 

bchain/coins/dcr/decredrpc.go

@@ -54,7 +54,7 @@ func NewDecredRPC(config json.RawMessage, pushHandler func(bchain.NotificationTy
 		Dial:                (&net.Dialer{KeepAlive: 600 * time.Second}).Dial,
 		MaxIdleConns:        100,
 		MaxIdleConnsPerHost: 100, // necessary to not to deplete ports
-		TLSClientConfig: &tls.Config{InsecureSkipVerify: true},
+		TLSClientConfig:     &tls.Config{InsecureSkipVerify: true},
 	}
 
 	d := &DecredRPC{

bchain/coins/deeponion/deeponionparser_test.go

@@ -1,4 +1,4 @@
-// +build unittest
+//go:build unittest
 
 package deeponion
 

bchain/coins/digibyte/digibyteparser.go

@@ -6,14 +6,14 @@ import (
 	"github.com/trezor/blockbook/bchain/coins/btc"
 )
 
+// network constants
 const (
-	// MainnetMagic is mainnet network constant
 	MainnetMagic wire.BitcoinNet = 0xdab6c3fa
 	TestnetMagic wire.BitcoinNet = 0xddbdc8fd
 )
 
+// parser parameters
 var (
-	// MainNetParams are parser parameters for mainnet
 	MainNetParams chaincfg.Params
 	TestNetParams chaincfg.Params
 )

bchain/coins/digibyte/digibyteparser_test.go

@@ -1,4 +1,4 @@
-// +build unittest
+//go:build unittest
 
 package digibyte
 

bchain/coins/divi/diviparser_test.go

@@ -1,4 +1,4 @@
-// +build unittest
+//go:build unittest
 
 package divi
 

bchain/coins/dogecoin/dogecoinparser_test.go

@@ -1,4 +1,4 @@
-// +build unittest
+//go:build unittest
 
 package dogecoin
 

bchain/coins/eth/erc20_test.go

@@ -1,4 +1,4 @@
-// +build unittest
+//go:build unittest
 
 package eth
 

bchain/coins/eth/ethparser_test.go

@@ -1,4 +1,4 @@
-// +build unittest
+//go:build unittest
 
 package eth
 

bchain/coins/firo/firoparser.go

@@ -26,11 +26,11 @@ const (
 	TestnetMagic wire.BitcoinNet = 0xcffcbeea
 	RegtestMagic wire.BitcoinNet = 0xfabfb5da
 
-	GenesisBlockTime       = 1414776286
+	GenesisBlockTime                  = 1414776286
-	SwitchToMTPBlockHeader = 1544443200
+	SwitchToMTPBlockHeader            = 1544443200
 	SwitchToProgPowBlockHeaderTestnet = 1630069200
 	SwitchToProgPowBlockHeaderMainnet = 1635228000
-	MTPL                   = 64
+	MTPL                              = 64
 
 	SpendTxID = "0000000000000000000000000000000000000000000000000000000000000000"
 
@@ -159,7 +159,7 @@ func (p *FiroParser) ParseBlock(b []byte) (*bchain.Block, error) {
 	}
 
 	// then ProgPow or MTP header
-	if(isProgPow(header, p.Params.Net == TestnetMagic)){
+	if isProgPow(header, p.Params.Net == TestnetMagic) {
 		progPowHeader := ProgPowBlockHeader{}
 
 		// header

bchain/coins/firo/firoparser_test.go

@@ -1,4 +1,4 @@
-// +build unittest
+//go:build unittest
 
 package firo
 

bchain/coins/flo/floparser_test.go

@@ -1,4 +1,4 @@
-// +build unittest
+//go:build unittest
 
 package flo
 

bchain/coins/fujicoin/fujicoinparser_test.go

@@ -1,4 +1,4 @@
-// +build unittest
+//go:build unittest
 
 package fujicoin
 

bchain/coins/gamecredits/gamecreditsparser_test.go

@@ -1,4 +1,4 @@
-// +build unittest
+//go:build unittest
 
 package gamecredits
 

bchain/coins/grs/grsparser.go

@@ -50,7 +50,7 @@ type GroestlcoinParser struct {
 func NewGroestlcoinParser(params *chaincfg.Params, c *btc.Configuration) *GroestlcoinParser {
 	return &GroestlcoinParser{
 		BitcoinParser: btc.NewBitcoinParser(params, c),
-		baseparser:        &bchain.BaseParser{},
+		baseparser:    &bchain.BaseParser{},
 	}
 }
 

bchain/coins/grs/grsparser_test.go

@@ -1,4 +1,4 @@
-// +build unittest
+//go:build unittest
 
 package grs
 

bchain/coins/koto/kotoparser_test.go

@@ -1,4 +1,4 @@
-// +build unittest
+//go:build unittest
 
 package koto
 

bchain/coins/liquid/liquidparser_test.go

@@ -1,4 +1,4 @@
-// +build unittest
+//go:build unittest
 
 package liquid
 

bchain/coins/litecoin/litecoinparser_test.go

@@ -1,4 +1,4 @@
-// +build unittest
+//go:build unittest
 
 package litecoin
 

bchain/coins/monacoin/monacoinparser_test.go

@@ -1,4 +1,4 @@
-// +build unittest
+//go:build unittest
 
 package monacoin
 

bchain/coins/monetaryunit/monetaryunitparser_test.go

@@ -1,4 +1,4 @@
-// +build unittest
+//go:build unittest
 
 package monetaryunit
 

bchain/coins/myriad/myriadparser_test.go

@@ -1,4 +1,4 @@
-// +build unittest
+//go:build unittest
 
 package myriad
 

bchain/coins/namecoin/namecoinparser_test.go

@@ -1,4 +1,4 @@
-// +build unittest
+//go:build unittest
 
 package namecoin
 

bchain/coins/nuls/nulsparser_test.go

@@ -1,5 +1,4 @@
 //go:build unittest
-// +build unittest
 
 package nuls
 

bchain/coins/omotenashicoin/omotenashicoinparser_test.go

@@ -1,4 +1,4 @@
-// +build unittest
+//go:build unittest
 
 package omotenashicoin
 

bchain/coins/pivx/pivxparser_test.go

@@ -1,4 +1,4 @@
-// +build unittest
+//go:build unittest
 
 package pivx
 

bchain/coins/polis/polisparser_test.go

@@ -1,4 +1,4 @@
-// +build unittest
+//go:build unittest
 
 package polis
 

bchain/coins/qtum/qtumparser_test.go

@@ -1,4 +1,4 @@
-// +build unittest
+//go:build unittest
 
 package qtum
 

bchain/coins/ravencoin/ravencoinparser_test.go

@@ -1,4 +1,4 @@
-// +build unittest
+//go:build unittest
 
 package ravencoin
 

bchain/coins/ritocoin/ritocoinparser_test.go

@@ -1,4 +1,4 @@
-// +build unittest
+//go:build unittest
 
 package ritocoin
 

bchain/coins/snowgem/snowgemparser_test.go

@@ -1,4 +1,4 @@
-// +build unittest
+//go:build unittest
 
 package snowgem
 

bchain/coins/trezarcoin/trezarcoinparser_test.go

@@ -1,4 +1,4 @@
-// +build unittest
+//go:build unittest
 
 package trezarcoin
 

bchain/coins/unobtanium/unobtaniumparser_test.go

@@ -1,4 +1,4 @@
-// +build unittest
+//go:build unittest
 
 package unobtanium
 

bchain/coins/vertcoin/vertcoinparser_test.go

@@ -1,4 +1,4 @@
-// +build unittest
+//go:build unittest
 
 package vertcoin
 

bchain/coins/viacoin/viacoinparser_test.go

@@ -1,4 +1,4 @@
-// +build unittest
+//go:build unittest
 
 package viacoin
 

bchain/coins/vipstarcoin/vipstarcoinparser_test.go

@@ -1,4 +1,4 @@
-// +build unittest
+//go:build unittest
 
 package vipstarcoin
 

bchain/coins/zec/zcashparser_test.go

@@ -1,4 +1,4 @@
-// +build unittest
+//go:build unittest
 
 package zec
 

bchain/coins/zec/zcashrpc.go

@@ -15,6 +15,7 @@ type ZCashRPC struct {
 	*btc.BitcoinRPC
 }
 
+// ResGetBlockChainInfo is a response to GetChainInfo request
 type ResGetBlockChainInfo struct {
 	Error  *bchain.RPCError `json:"error"`
 	Result struct {
@@ -72,6 +73,7 @@ func (z *ZCashRPC) Initialize() error {
 	return nil
 }
 
+// GetChainInfo return info about the blockchain
 func (z *ZCashRPC) GetChainInfo() (*bchain.ChainInfo, error) {
 	chainInfo := ResGetBlockChainInfo{}
 	err := z.Call(&btc.CmdGetBlockChainInfo{Method: "getblockchaininfo"}, &chainInfo)

build/tools/templates.go

@@ -12,6 +12,7 @@ import (
 	"time"
 )
 
+// Config contains the structure of the config
 type Config struct {
 	Coin struct {
 		Name     string `json:"name"`
@@ -108,6 +109,7 @@ func generateRPCAuth(user, pass string) (string, error) {
 	return out.String(), nil
 }
 
+// ParseTemplate parses the template
 func (c *Config) ParseTemplate() *template.Template {
 	templates := map[string]string{
 		"IPC.RPCURLTemplate":                      c.IPC.RPCURLTemplate,
@@ -134,6 +136,7 @@ func (c *Config) ParseTemplate() *template.Template {
 	return t
 }
 
+// LoadConfig loads the config files
 func LoadConfig(configsDir, coin string) (*Config, error) {
 	config := new(Config)
 
@@ -191,6 +194,7 @@ func isEmpty(config *Config, target string) bool {
 	}
 }
 
+// GeneratePackageDefinitions generate the package definitions from the config
 func GeneratePackageDefinitions(config *Config, templateDir, outputDir string) error {
 	templ := config.ParseTemplate()
 
@@ -276,12 +280,7 @@ func writeTemplate(path string, info os.FileInfo, templ *template.Template, conf
 	}
 	defer f.Close()
 
-	err = templ.ExecuteTemplate(f, "main", config)
+	return templ.ExecuteTemplate(f, "main", config)
-	if err != nil {
-		return err
-	}
-
-	return nil
 }
 
 func writeBackendServerConfigFile(config *Config, outputDir string) error {
@@ -318,18 +317,13 @@ func writeBackendClientConfigFile(config *Config, outputDir string) error {
 
 	if config.Backend.ClientConfigFile == "" {
 		return nil
-	} else {
-		in, err := os.Open(filepath.Join(outputDir, "backend/config", config.Backend.ClientConfigFile))
-		if err != nil {
-			return err
-		}
-		defer in.Close()
-
-		_, err = io.Copy(out, in)
-		if err != nil {
-			return err
-		}
 	}
+	in, err := os.Open(filepath.Join(outputDir, "backend/config", config.Backend.ClientConfigFile))
+	if err != nil {
+		return err
+	}
+	defer in.Close()
 
-	return nil
+	_, err = io.Copy(out, in)
+	return err
 }

common/jsonnumber_test.go

@@ -1,4 +1,4 @@
-// +build unittest
+//go:build unittest
 
 package common
 

db/rocksdb_ethereumtype_test.go

@@ -1,4 +1,4 @@
-// +build unittest
+//go:build unittest
 
 package db
 

db/rocksdb_test.go

@@ -1,4 +1,4 @@
-// +build unittest
+//go:build unittest
 
 package db
 

db/test_helper.go

@@ -1,4 +1,4 @@
-// +build integration
+//go:build integration
 
 package db
 

fiat/fiat_rates_test.go

@@ -1,4 +1,4 @@
-// +build unittest
+//go:build unittest
 
 package fiat
 

server/public_test.go

@@ -1,5 +1,4 @@
 //go:build unittest
-// +build unittest
 
 package server
 

server/socketio_log_test.go

@@ -1,4 +1,4 @@
-// +build integration
+//go:build integration
 
 package server
 

tests/integration.go

@@ -1,4 +1,4 @@
-// +build integration
+//go:build integration
 
 package tests
 

tests/integration_test.go

@@ -1,4 +1,4 @@
-// +build integration
+//go:build integration
 
 package tests
 

tests/rpc/rpc.go

@@ -1,4 +1,4 @@
-// +build integration
+//go:build integration
 
 package rpc
 

tests/sync/connectblocks.go

@@ -1,4 +1,4 @@
-// +build integration
+//go:build integration
 
 package sync
 

tests/sync/fakechain.go

@@ -1,4 +1,4 @@
-// +build integration
+//go:build integration
 
 package sync
 

tests/sync/handlefork.go

@@ -1,4 +1,4 @@
-// +build integration
+//go:build integration
 
 package sync
 

tests/sync/sync.go

@@ -1,4 +1,4 @@
-// +build integration
+//go:build integration
 
 package sync