Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test: add testing against Go 1.20 RC #250

Merged
merged 2 commits into from
Dec 12, 2022
Merged

Conversation

tri-adam
Copy link
Member

@tri-adam tri-adam commented Dec 9, 2022

Remove ECDSA usage from Test_dsseEncoder_signMessage, since signatures generated with the crypto/ecdsa package are not deterministic. Start testing against Go 1.20 RCs in CI.

@tri-adam tri-adam self-assigned this Dec 9, 2022
@codecov-commenter
Copy link

Codecov Report

Merging #250 (366218c) into main (0ffd7e3) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #250   +/-   ##
=======================================
  Coverage   72.78%   72.78%           
=======================================
  Files          35       35           
  Lines        2862     2862           
=======================================
  Hits         2083     2083           
  Misses        626      626           
  Partials      153      153           

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

@tri-adam tri-adam force-pushed the go-1.20rc branch 2 times, most recently from dc2fdb5 to 66958b2 Compare December 9, 2022 19:13
Signatures generated by the crypto/ecdsa package are not deterministic,
so the golden file pattern should not be used with ECDSA keys.
@tri-adam tri-adam marked this pull request as ready for review December 9, 2022 19:19
@tri-adam tri-adam merged commit 8905048 into sylabs:main Dec 12, 2022
@tri-adam tri-adam deleted the go-1.20rc branch December 12, 2022 14:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants