chore: add support for funcorder

Author: remyleone

.golangci.yml

@@ -29,6 +29,7 @@ linters:
     - exptostd # Detects functions from golang.org/x/exp/ that can be replaced by std functions. [auto-fix]
     - fatcontext # Detects nested contexts in loops and function literals. [auto-fix]
     - forbidigo # Forbids identifiers [fast: true, auto-fix: false]
+    - funcorder # Checks the order of functions, methods, and constructors. [fast]
     - gocheckcompilerdirectives # Checks that go compiler directive comments (//go:) are valid. [fast: true, auto-fix: false]
     - gochecksumtype # Run exhaustiveness checks on Go "sum types" [fast: false, auto-fix: false]
     - goconst # Finds repeated strings that could be replaced by a constant [fast: true, auto-fix: false]

internal/locality/regional/ids.go

@@ -14,17 +14,17 @@ type ID struct {
 	Region scw.Region
 }
 
-func (z ID) String() string {
-	return fmt.Sprintf("%s/%s", z.Region, z.ID)
-}
-
 func NewID(region scw.Region, id string) ID {
 	return ID{
 		ID:     id,
 		Region: region,
 	}
 }
 
+func (z ID) String() string {
+	return fmt.Sprintf("%s/%s", z.Region, z.ID)
+}
+
 func ExpandID(id interface{}) ID {
 	regionalID := ID{}
 	tab := strings.Split(id.(string), "/")

internal/locality/zonal/ids.go

@@ -14,17 +14,17 @@ type ID struct {
 	Zone scw.Zone
 }
 
-func (z ID) String() string {
-	return fmt.Sprintf("%s/%s", z.Zone, z.ID)
-}
-
 func NewID(zone scw.Zone, id string) ID {
 	return ID{
 		ID:   id,
 		Zone: zone,
 	}
 }
 
+func (z ID) String() string {
+	return fmt.Sprintf("%s/%s", z.Zone, z.ID)
+}
+
 func ExpandID(id interface{}) ID {
 	zonedID := ID{}
 	tab := strings.Split(id.(string), "/")

internal/services/instance/instancehelpers/block.go

@@ -11,16 +11,6 @@ import (
 	"github.com/scaleway/terraform-provider-scaleway/v2/internal/meta"
 )
 
-func NewBlockAndInstanceAPI(client *scw.Client) *BlockAndInstanceAPI {
-	instanceAPI := instance.NewAPI(client)
-	blockAPI := block.NewAPI(client)
-
-	return &BlockAndInstanceAPI{
-		API:      instanceAPI,
-		BlockAPI: blockAPI,
-	}
-}
-
 // InstanceAndBlockAPIWithZone returns a new instance API and the zone for a Create request
 func InstanceAndBlockAPIWithZone(d *schema.ResourceData, m interface{}) (*BlockAndInstanceAPI, scw.Zone, error) {
 	zone, err := meta.ExtractZone(d, m)
@@ -46,6 +36,16 @@ type BlockAndInstanceAPI struct {
 	BlockAPI *block.API
 }
 
+func NewBlockAndInstanceAPI(client *scw.Client) *BlockAndInstanceAPI {
+	instanceAPI := instance.NewAPI(client)
+	blockAPI := block.NewAPI(client)
+
+	return &BlockAndInstanceAPI{
+		API:      instanceAPI,
+		BlockAPI: blockAPI,
+	}
+}
+
 type GetUnknownVolumeRequest struct {
 	VolumeID string
 	Zone     scw.Zone

internal/workerpool/workerpool.go

@@ -27,6 +27,31 @@ func NewWorkerPool(size int) *WorkerPool {
 	return p
 }
 
+func (p *WorkerPool) AddTask(task Task) {
+	p.tasksWaitingGroup.Add(1)
+	p.tasksToDispatch <- task
+}
+
+func (p *WorkerPool) CloseAndWait() []error {
+	close(p.tasksToDispatch)
+	p.tasksWaitingGroup.Wait()
+
+	return p.errors
+}
+
+func (p *WorkerPool) worker() {
+	for task := range p.tasksToRun {
+		err := task()
+		if err != nil {
+			p.errorsMutex.Lock()
+			p.errors = append(p.errors, err)
+			p.errorsMutex.Unlock()
+		}
+
+		p.tasksWaitingGroup.Done()
+	}
+}
+
 func (p *WorkerPool) dispatcher() {
 	var pendingTasks []Task
 
@@ -60,28 +85,3 @@ func (p *WorkerPool) dispatcher() {
 		}
 	}
 }
-
-func (p *WorkerPool) worker() {
-	for task := range p.tasksToRun {
-		err := task()
-		if err != nil {
-			p.errorsMutex.Lock()
-			p.errors = append(p.errors, err)
-			p.errorsMutex.Unlock()
-		}
-
-		p.tasksWaitingGroup.Done()
-	}
-}
-
-func (p *WorkerPool) AddTask(task Task) {
-	p.tasksWaitingGroup.Add(1)
-	p.tasksToDispatch <- task
-}
-
-func (p *WorkerPool) CloseAndWait() []error {
-	close(p.tasksToDispatch)
-	p.tasksWaitingGroup.Wait()
-
-	return p.errors
-}