dedis / kyber

Advanced crypto library for the Go language
Other
643 stars 168 forks source link

CI tests for 32bit correction + version of Go update #526

Closed matteosz closed 5 months ago

matteosz commented 5 months ago

It addresses @ineiti's comments of using a matrix to simplify the ci.

Resolves #522

github-actions[bot] commented 5 months ago

🔒 Could not start CI tests due to missing safe PR label. Please contact a DEDIS maintainer.

github-actions[bot] commented 5 months ago

🔒 Could not start CI tests due to missing safe PR label. Please contact a DEDIS maintainer.

ineiti commented 5 months ago

What is your reasoning behind testing go 1.18 and 1.22?

In some past projects we had a policy of "the last two stable versions". You might fail on go 1.19, 1.20, or 1.21 with your policy.

sonarcloud[bot] commented 5 months ago

Quality Gate Failed Quality Gate failed

Failed conditions
37.7% Duplication on New Code (required ≤ 10%)

See analysis details on SonarCloud