upgrade to go 1.20.1

[cpanato]

Summary

• upgrade to go 1.20.1

[haydentherapper]

Thank you!

[haydentherapper]

Ah, one unit test failure, looks like you need a different way to seed rand?

[haydentherapper]

Oh you probably need to update the expected test results, I guess the new implementation of random doesn't result in a the same deterministic order (which is kinda strange if true?)

[cpanato]

Ah, one unit test failure, looks like you need a different way to seed rand?

I saw, did not have time yesterday to fix it, will try this weekend

[codecov-commenter]

Codecov Report

Merging #245 (36829d7) into main (8110f91) will decrease coverage by 0.14%.
The diff coverage is 28.57%.

@@            Coverage Diff             @@
##             main     #245      +/-   ##
==========================================
- Coverage   52.66%   52.53%   -0.14%     
==========================================
  Files          19       19              
  Lines        1143     1146       +3     
==========================================
  Hits          602      602              
- Misses        483      486       +3     
  Partials       58       58              

Impacted Files	Coverage Δ
pkg/ntpmonitor/ntpmonitor.go	62.93% <0.00%> (-1.68%)	⬇️
pkg/ntpmonitor/randomchoice.go	100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[cpanato]

@haydentherapper let me know if you agree with the rand change. PTAL

[cpanato]

@haydentherapper PTAL

[haydentherapper]

@cpanato could you rebase, then the tests should pass?

[cpanato]

@haydentherapper done, PTAL

[haydentherapper]

Thank you!