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/meta/meta.go

@@ -45,45 +45,6 @@ type Meta struct {
 	credentialsSource *CredentialsSource
 }
 
-func (m Meta) ScwClient() *scw.Client {
-	return m.scwClient
-}
-
-func (m Meta) HTTPClient() *http.Client {
-	return m.httpClient
-}
-
-func (m Meta) AccessKeySource() string {
-	return m.credentialsSource.AccessKey
-}
-
-func (m Meta) SecretKeySource() string {
-	return m.credentialsSource.SecretKey
-}
-
-func (m Meta) ProjectIDSource() string {
-	return m.credentialsSource.ProjectID
-}
-
-func (m Meta) RegionSource() string {
-	return m.credentialsSource.DefaultRegion
-}
-
-func (m Meta) ZoneSource() string {
-	return m.credentialsSource.DefaultZone
-}
-
-type Config struct {
-	ProviderSchema      *schema.ResourceData
-	HTTPClient          *http.Client
-	TerraformVersion    string
-	ForceZone           scw.Zone
-	ForceProjectID      string
-	ForceOrganizationID string
-	ForceAccessKey      string
-	ForceSecretKey      string
-}
-
 // NewMeta creates the Meta object containing the SDK client.
 func NewMeta(ctx context.Context, config *Config) (*Meta, error) {
 	////
@@ -149,6 +110,45 @@ func NewMeta(ctx context.Context, config *Config) (*Meta, error) {
 	}, nil
 }
 
+func (m Meta) ScwClient() *scw.Client {
+	return m.scwClient
+}
+
+func (m Meta) HTTPClient() *http.Client {
+	return m.httpClient
+}
+
+func (m Meta) AccessKeySource() string {
+	return m.credentialsSource.AccessKey
+}
+
+func (m Meta) SecretKeySource() string {
+	return m.credentialsSource.SecretKey
+}
+
+func (m Meta) ProjectIDSource() string {
+	return m.credentialsSource.ProjectID
+}
+
+func (m Meta) RegionSource() string {
+	return m.credentialsSource.DefaultRegion
+}
+
+func (m Meta) ZoneSource() string {
+	return m.credentialsSource.DefaultZone
+}
+
+type Config struct {
+	ProviderSchema      *schema.ResourceData
+	HTTPClient          *http.Client
+	TerraformVersion    string
+	ForceZone           scw.Zone
+	ForceProjectID      string
+	ForceOrganizationID string
+	ForceAccessKey      string
+	ForceSecretKey      string
+}
+
 func customizeUserAgent(providerVersion string, terraformVersion string) string {
 	userAgent := fmt.Sprintf("terraform-provider/%s terraform/%s", providerVersion, terraformVersion)
 

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
-}