Repository navigation
Update Go to 1.27 and refresh tooling and dependencies - #141
Merged
Merged
Conversation
Use Go 1.27 for builds and refresh the checkout, setup-go and upload-assets actions. Signed-off-by: Han Verstraete (OpenFaaS Ltd) <han@openfaas.com>
Refresh cloud provider SDKs, authentication libraries and supporting modules. Regenerate the vendor tree. Signed-off-by: Han Verstraete (OpenFaaS Ltd) <han@openfaas.com>
AI Pull Request OverviewDisclaimer: This review was generated by automated AI and may contain errors. Do not trust its outputs without human verification. Summary
Approval rating (1-10)9/10. The toolchain, workflows, dependency metadata, and regenerated dependency snapshot are coherent, with no concrete regression identified in the scoped changes. Summary per fileSummary per file
Overall AssessmentThe final change set is coherent and ready to merge. The declared Go version matches both workflows, dependency metadata is synchronized with the regenerated module snapshot, and the reviewed direct API usage remains compatible with the selected versions. No actionable defect was found. Detailed ReviewDetailed ReviewNo findings. AI agent details. |
welteki
marked this pull request as ready for review
October 22, 2026 15:38
This file contains hidden or 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
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.
Description
Use Go 1.27 in the module and Go 1.27.x in both workflows. Update checkout and setup-go to v7 and upload-assets to 0.5.0.
Refresh the cloud provider SDKs, authentication libraries, password generator and supporting dependencies, and regenerate the vendor tree.
How Has This Been Tested?
Validated on Linux amd64 using Go 1.27.3 with
GOMAXPROCS=2 GOFLAGS=-p=1to limit memory use:make all: formatting checks, all unit tests, six release builds, archive generation and checksums.go mod verify: all modules verified.go vet ./...: passes.Native
inletsctl version,create --help, anddelete --helpsmoke checks pass.All six release targets built: Linux amd64/arm64/arm (GOARM=6), Darwin amd64/arm64, and Windows amd64.
file bin/inletsctl*confirms ELF, Mach-O and PE32+ formats;go version -m bin/inletsctl-armhfconfirms GOARM=6.Verified every generated SHA-256 checksum and archive payload against its binary.
Cross-platform binaries were not executed. No live cloud provisioning was tested. There are no Dockerfiles or Docker release platforms in this repository.
How are existing users impacted? What migration steps/scripts do we need?
Building from source requires Go 1.27. Existing CLI usage is unchanged; no migration steps are required.
Checklist:
I have:
git commit -s