Fix a bunch of misspellings in our Go code (all in comments).

Thank you https://goreportcard.com/report/github.com/curoverse/arvados#misspell

No issue #

diff --git a/sdk/go/manifest/manifest.go b/sdk/go/manifest/manifest.go
index cf0ae85c5782a8848d234de10d3fce81b6e661cd..22b1c974e634cd8229b645421ecb09480807c000 100644 (file)
--- a/sdk/go/manifest/manifest.go
+++ b/sdk/go/manifest/manifest.go
@@ -265,7 +265,7 @@ func (m *Manifest) FileSegmentIterByName(filepath string) <-chan *FileSegment {
        return ch
 }
 
-// Blocks may appear mulitple times within the same manifest if they
+// Blocks may appear multiple times within the same manifest if they
 // are used by multiple files. In that case this Iterator will output
 // the same block multiple times.
 //

diff --git a/sdk/go/streamer/transfer.go b/sdk/go/streamer/transfer.go
index 3f5f9344c521f3021e2f6fd35f025e4b9c7fb95e..499b4d966810223d7e1747caff4a1ce39113b470 100644 (file)
--- a/sdk/go/streamer/transfer.go
+++ b/sdk/go/streamer/transfer.go
@@ -16,13 +16,13 @@ channel back to the transfer() function.
 Meanwhile, the transfer() function selects() on two channels, the "requests"
 channel and the "slices" channel.
 
-When a message is recieved on the "slices" channel, this means the a new
+When a message is received on the "slices" channel, this means the a new
 section of the buffer has data, or an error is signaled.  Since the data has
 been read directly into the source_buffer, it is able to simply increases the
 size of the body slice to encompass the newly filled in section.  Then any
 pending reads are serviced with handleReadRequest (described below).
 
-When a message is recieved on the "requests" channel, it means a StreamReader
+When a message is received on the "requests" channel, it means a StreamReader
 wants access to a slice of the buffer.  This is passed to handleReadRequest().
 
 The handleReadRequest() function takes a sliceRequest consisting of a buffer

diff --git a/services/crunch-run/crunchrun_test.go b/services/crunch-run/crunchrun_test.go
index c826a21e74e08405ab6bca75d02716cdad1fb079..5ee879d9f99c1c59aa58bbf4f951fa28ed4c0d98 100644 (file)
--- a/services/crunch-run/crunchrun_test.go
+++ b/services/crunch-run/crunchrun_test.go
@@ -452,7 +452,7 @@ func (s *TestSuite) TestUpdateContainerRecordCancelled(c *C) {
 }
 
 // Used by the TestFullRun*() test below to DRY up boilerplate setup to do full
-// dress rehersal of the Run() function, starting from a JSON container record.
+// dress rehearsal of the Run() function, starting from a JSON container record.
 func FullRunHelper(c *C, record string, fn func(t *TestDockerClient)) (api *ArvTestClient, cr *ContainerRunner) {
        rec := ContainerRecord{}
        err := json.NewDecoder(strings.NewReader(record)).Decode(&rec)

diff --git a/services/datamanager/loggerutil/loggerutil.go b/services/datamanager/loggerutil/loggerutil.go
index 8c655cd5ff68a981146493bfa21fd71693ba12c0..8111425d7af76a9fa55ca71a3b3620a1d0a095a1 100644 (file)
--- a/services/datamanager/loggerutil/loggerutil.go
+++ b/services/datamanager/loggerutil/loggerutil.go
@@ -10,7 +10,7 @@ import (
        "time"
 )
 
-// Useful to call at the begining of execution to log info about the
+// Useful to call at the beginning of execution to log info about the
 // current run.
 func LogRunInfo(arvLogger *logger.Logger) {
        if arvLogger != nil {

diff --git a/services/datamanager/summary/pull_list.go b/services/datamanager/summary/pull_list.go
index aac9aec9716a74c33f5c18a4ec23e2f9b0b8e212..d7fb3eb8f7cb3953d0f40e0e67626376f0d2941a 100644 (file)
--- a/services/datamanager/summary/pull_list.go
+++ b/services/datamanager/summary/pull_list.go
@@ -71,7 +71,7 @@ func ComputePullServers(kc *keepclient.KeepClient,
        blockToDesiredReplication map[blockdigest.DigestWithSize]int,
        underReplicated BlockSet) (m map[Locator]PullServers) {
        m = map[Locator]PullServers{}
-       // We use CanonicalString to avoid filling memory with dupicate
+       // We use CanonicalString to avoid filling memory with duplicate
        // copies of the same string.
        var cs CanonicalString
 

diff --git a/services/keepproxy/keepproxy.go b/services/keepproxy/keepproxy.go
index 7b5cd2befb8f69bd25fa62674d01590214aec5ad..4cd931037ef830dfd8a6b25022126c84c13d7036 100644 (file)
--- a/services/keepproxy/keepproxy.go
+++ b/services/keepproxy/keepproxy.go
@@ -185,7 +185,7 @@ func CheckAuthorizationHeader(kc *keepclient.KeepClient, cache *ApiTokenCache, r
        }
 
        if cache.RecallToken(tok) {
-               // Valid in the cache, short circut
+               // Valid in the cache, short circuit
                return true, tok
        }
 

diff --git a/services/keepstore/trash_worker_test.go b/services/keepstore/trash_worker_test.go
index 1d3063a9de10651cf675062a439403372d39f49f..ac9406178c00ffaffc024f61ffc981479fc9d4e9 100644 (file)
--- a/services/keepstore/trash_worker_test.go
+++ b/services/keepstore/trash_worker_test.go
@@ -163,7 +163,7 @@ func TestTrashWorkerIntegration_TwoDifferentLocatorsInVolume1(t *testing.T) {
 }
 
 /* Allow default Trash Life time to be used. Thus, the newly created block
-   will not be deleted becuase its Mtime is within the trash life time.
+   will not be deleted because its Mtime is within the trash life time.
 */
 func TestTrashWorkerIntegration_SameLocatorInTwoVolumesWithDefaultTrashLifeTime(t *testing.T) {
        neverDelete = false