chore: update `github.com/cespare/xxhash` to v2 #16094

Juneezee · 2025-01-10T15:39:29Z

github.com/cespare/xxhash/v2 is the latest version with bug fixes and improvements. No breaking changes, see https://go.dev/play/p/4fisKvMhrGR.

Affected lines of code are:

coder/provisioner/terraform/timings.go

Lines 202 to 205 in 8b9763d

    
           func (e *timingSpan) hashByState(state proto.TimingState) uint64 { 
        
           	id := fmt.Sprintf("%s:%s:%s:%s:%s", e.kind.Category(), state.String(), e.action, e.resource, e.provider) 
        
           	return xxhash.Sum64String(id) 
        
           }

coder/provisioner/terraform/internal/timings_test_utils.go

Lines 90 to 91 in 8b9763d

ah := xxhash.Sum64String(a.String())

bh := xxhash.Sum64String(b.String())

`github.com/cespare/xxhash/v2` is the latest version with bug fixes and improvements. No breaking changes, see https://go.dev/play/p/4fisKvMhrGR. Signed-off-by: Eng Zer Jun <[email protected]>

dannykopping

LGTM, thanks for your contribution @Juneezee 👍

chore: update github.com/cespare/xxhash to v2

35f0553

`github.com/cespare/xxhash/v2` is the latest version with bug fixes and improvements. No breaking changes, see https://go.dev/play/p/4fisKvMhrGR. Signed-off-by: Eng Zer Jun <[email protected]>

cdr-bot bot added the community Pull Requests and issues created by the community. label Jan 10, 2025

github-actions bot assigned Juneezee Jan 10, 2025

matifali requested review from spikecurtis and dannykopping January 15, 2025 14:55

dannykopping approved these changes Jan 15, 2025

View reviewed changes

dannykopping merged commit 92d02ad into coder:main Jan 15, 2025
33 checks passed

github-actions bot locked and limited conversation to collaborators Jan 15, 2025

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

chore: update `github.com/cespare/xxhash` to v2 #16094

chore: update `github.com/cespare/xxhash` to v2 #16094

Juneezee commented Jan 10, 2025

dannykopping left a comment

	func (e *timingSpan) hashByState(state proto.TimingState) uint64 {
	id := fmt.Sprintf("%s:%s:%s:%s:%s", e.kind.Category(), state.String(), e.action, e.resource, e.provider)
	return xxhash.Sum64String(id)
	}

	ah := xxhash.Sum64String(a.String())
	bh := xxhash.Sum64String(b.String())

chore: update github.com/cespare/xxhash to v2 #16094

chore: update github.com/cespare/xxhash to v2 #16094

Conversation

Juneezee commented Jan 10, 2025

dannykopping left a comment

Choose a reason for hiding this comment

chore: update `github.com/cespare/xxhash` to v2 #16094

chore: update `github.com/cespare/xxhash` to v2 #16094