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

Add job to evaluate Go 1.22 loop behavior changes #180

Merged
merged 1 commit into from
Feb 22, 2024

Conversation

atc0005
Copy link
Owner

@atc0005 atc0005 commented Feb 22, 2024

Use Go 1.21 series "mirror" image to perform builds using the loopvar compiler flag. This new workflow job will be ued to identify loops that compile differently using the new loop semantics introduced as a preview in Go 1.21 and enabled by default with the release of Go 1.22.

This new workflow job is set to continue-on-error in order to verify that it works as intended. Once it has been proven to work as intended continue-on-error will be set to false and any identified loop behavior changes will be considered a real CI failure requiring resolution.

refs GH-145

Use Go 1.21 series "mirror" image to perform builds using the
`loopvar` compiler flag. This new workflow job will be ued to
identify loops that compile differently using the new loop
semantics introduced as a preview in Go 1.21 and enabled by
default with the release of Go 1.22.

This new workflow job is set to `continue-on-error` in order to
verify that it works as intended. Once it has been proven to
work as intended `continue-on-error` will be set to `false`
and any identified loop behavior changes will be considered
a real CI failure requiring resolution.

refs GH-145
@atc0005 atc0005 self-assigned this Feb 22, 2024
@atc0005 atc0005 merged commit 4b8b5f0 into master Feb 22, 2024
3 checks passed
@atc0005 atc0005 deleted the i145-add-job-to-evaluate-for-loop-behavior-changes branch February 22, 2024 13:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant