Review feedback

Author: luxas

.golangci.yml

@@ -46,7 +46,7 @@ linters:
   # - godox don't error although there are TODOs in the code
   - funlen
   - whitespace
-  # - wsl allow "non-ideomatic" whitespace formattings for now
+  # - wsl allow "non-idiomatic" whitespace formattings for now
   - goprintffuncname
   - gomnd
   - goerr113

bitbucket/doc.go

@@ -17,6 +17,6 @@ limitations under the License.
 package bitbucket
 
 import (
-	// Dummy import until we have the implementation ready.
+	// TODO: Dummy import until we have the implementation ready.
 	_ "github.com/ktrysmt/go-bitbucket"
 )

github/resource_repository.go

@@ -136,7 +136,7 @@ func (r *userRepository) Reconcile(ctx context.Context) (bool, error) {
 	return true, r.Update(ctx)
 }
 
-// Delete deletes the current resource irreversible.
+// Delete deletes the current resource irreversibly.
 //
 // ErrNotFound is returned if the resource doesn't exist anymore.
 func (r *userRepository) Delete(ctx context.Context) error {

gitlab/doc.go

@@ -17,6 +17,6 @@ limitations under the License.
 package gitlab
 
 import (
-	// Dummy import until we have the implementation ready.
+	// TODO: Dummy import until we have the implementation ready.
 	_ "github.com/xanzy/go-gitlab"
 )

gitprovider/gitprovider.go

@@ -66,7 +66,7 @@ type Updatable interface {
 // Deletable is an interface which all objects that can be deleted
 // using the Client implement.
 type Deletable interface {
-	// Delete deletes the current resource irreversible.
+	// Delete deletes the current resource irreversibly.
 	//
 	// ErrNotFound is returned if the resource doesn't exist anymore.
 	Delete(ctx context.Context) error