-
Notifications
You must be signed in to change notification settings - Fork 18
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
bump deps #78
bump deps #78
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -14,7 +14,7 @@ jobs: | |
- name: Set up Go | ||
uses: actions/setup-go@v4 | ||
with: | ||
go-version: "1.22.5" | ||
go-version: "1.22" | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 💡 Codebase verification Go version specification needs alignment across workflows The verification reveals inconsistent Go version specifications across the project:
Recommendations:
🔗 Analysis chainVerify Go version consistency across the project. The Go version specification
While using 🏁 Scripts executedThe following scripts were executed for the analysis: Script: #!/bin/bash
# Description: Check Go version specifications across the project
# Expected: Consistent version requirements or documented differences
# Check Go version in workflow files
echo "Go versions in workflow files:"
rg -A 1 "go-version:" .github/workflows/
# Check Go version in README
echo -e "\nGo version in README:"
rg "Go v\d+\.\d+\.?\d*" README.md
# Check Go version in go.mod
echo -e "\nGo version in go.mod:"
rg "^go \d+\.\d+\.?\d*" go.mod
Length of output: 1041 |
||
- name: Set environment variables | ||
run: | | ||
MINIWASM_NETWORK_NAME="miniwasm-1" | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -12,7 +12,7 @@ jobs: | |
- name: Set up Go | ||
uses: actions/setup-go@v4 | ||
with: | ||
go-version: "1.22.5" | ||
go-version: "1.22" | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 💡 Codebase verification Go version mismatch across project files needs to be addressed The verification confirms version inconsistencies:
To ensure reproducible builds, update all workflow files:
🔗 Analysis chainEnsure consistent Go version across the project. The Go version
- go-version: "1.22"
+ go-version: "1.22.7"
🏁 Scripts executedThe following scripts were executed for the analysis: Script: #!/bin/bash
# Description: Check Go versions across workflow files
echo "Checking Go versions in workflow files..."
rg -A 1 "setup-go@v4" .github/workflows/
Length of output: 527 Script: #!/bin/bash
# Description: Check Go versions in all workflow files
echo "Go versions in workflow files:"
rg -A 2 "setup-go@v4" .github/workflows/
# Check go.mod version
echo -e "\nGo version in go.mod:"
rg "^go " go.mod
# Check README for Go version requirement
echo -e "\nGo version in README:"
rg -i "go.*[0-9]+\.[0-9]+\.[0-9]+" README.md
Length of output: 922 |
||
|
||
- name: Set environment variables | ||
run: | | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -23,14 +23,14 @@ concurrency: | |
jobs: | ||
golangci: | ||
env: | ||
GOLANGCI_LINT_VERSION: v1.59.1 | ||
GOLANGCI_LINT_VERSION: v1.61.0 | ||
name: golangci-lint | ||
runs-on: ubuntu-latest | ||
steps: | ||
- uses: actions/checkout@v4 | ||
- uses: actions/setup-go@v5 | ||
with: | ||
go-version: 1.22 | ||
go-version: 1.23 | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 💡 Codebase verification Go version 1.23 is incorrect and inconsistent with go.mod The go.mod file specifies Go 1.22.7, which conflicts with the Go 1.23 version specified in the workflow. Please update both occurrences in the lint.yml workflow to match the go.mod version: - go-version: 1.23
+ go-version: 1.22.7 🔗 Analysis chainCritical: Incorrect and inconsistent Go version specification There are several issues with the Go version changes:
Please update both occurrences to match the go.mod version: - go-version: 1.23
+ go-version: 1.22.7 Let's verify the Go versions across the repository: Also applies to: 75-75 🏁 Scripts executedThe following scripts were executed for the analysis: Script: #!/bin/bash
# Description: Check Go version specifications across the repository
# Expected: Consistent Go version specifications
echo "Checking Go versions in workflow files:"
rg -A 1 "setup-go@v5" .github/workflows/
echo -e "\nChecking Go version in go.mod:"
rg "^go \d+\.\d+(\.\d+)?" go.mod
Length of output: 458 |
||
check-latest: true | ||
- uses: technote-space/[email protected] | ||
id: git_diff | ||
|
@@ -72,7 +72,7 @@ jobs: | |
- name: Setup go | ||
uses: actions/setup-go@v5 | ||
with: | ||
go-version: 1.22 | ||
go-version: 1.23 | ||
check-latest: true | ||
- run: | | ||
go mod tidy | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Version specification should align with README requirements
The Go version is set to "1.22" which could potentially use an older patch version than the 1.22.7+ requirement mentioned in the README. This might lead to build inconsistencies.
Apply this change to ensure consistency with project requirements:
📝 Committable suggestion