Updated test matrix to latest go versions. #205
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
nickpaisley
force-pushed
the
NickP/PLAT-111508
branch
2 times, most recently
from
631a7d8 to
4215bfa
Compare
tomlongridge
approved these changes
Feb 9, 2024
nickpaisley
force-pushed
the
NickP/PLAT-111508
branch
from
9f2e0d7 to
80e0568
Compare
"build: added latest go versions to test matrix"
nickpaisley
force-pushed
the
NickP/PLAT-111508
branch
from
80e0568 to
615ab12
Compare
tomlongridge
approved these changes
Feb 14, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Goal
Update test matrix to include latest versions of Go.
Design
Make sure building with the latest versions of Go is successful and monitor how long it takes to build.
Changeset
Added latest Go versions,
v1.17 - v1.22, to test-matrix.Updated the
unhandledCrash()function inmain.gowithnoinlineto accommodate the inline function behavior introduced inv1.17+.Updated the
RUNcommands in all Dockerfile(s) to a conditional to usego getorgo installdependent on the version of Go being used.Moved
ARG GO_VERSIONup to allow consumption byRUNcommand.Testing
Relying on testing via CI.