-
Notifications
You must be signed in to change notification settings - Fork 3
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
Remove .go-version file #1040
Remove .go-version file #1040
Conversation
This commit removes the .go-version file, as the Go version is now explicitly defined in the go directive within the go.mod file. Reminder: Go 1.21 introduced toolchain management, i.e. your local go command can manually download and install the release indicated in the go.mod file. This is also leveraged by tools like vscode.
7f4c1ef
to
98453a1
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1040 +/- ##
=======================================
Coverage 55.35% 55.35%
=======================================
Files 103 103
Lines 6021 6021
=======================================
Hits 3333 3333
Misses 2435 2435
Partials 253 253 ☔ View full report in Codecov by Sentry. |
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.
Thanks!
uses: actions/setup-go@v5 | ||
with: | ||
go-version: ${{ steps.go_version.outputs.go_version }} | ||
go-version-file: go.mod |
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.
😍
@@ -104,7 +104,7 @@ Keycloack: | |||
| Dashboard | <http://localhost:8080> | `admin` | `admin123` | | |||
| MinIO console | <http://localhost:7460> | `admin` | `admin123` | | |||
| Temporal UI | <http://localhost:7440> | `admin` | `admin123` | | |||
| Keycloack | <http://localhost:7470> | `keycloack` | `keycloack123` | | |||
| Keycloak | <http://localhost:7470> | `keycloak` | `keycloak123` | |
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.
🙄
This PR removes the .go-version file, as the Go version is now explicitly
defined in the go directive within the go.mod file.
Reminder: Go 1.21 introduced toolchain management, i.e. your local go command
can manually download and install the release indicated in the go.mod file. This
is also leveraged by tools like vscode.