Update dependencies and improve project setup #41
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
Updated Go version and all module dependencies in go.mod for better compatibility and performance. Added an 'init-tools' makefile target to ensure all required tools for development are installed and properly set up. Adjusted 'lint' target to use local golangci-lint. Made minor code style adjustment in httptest.go. Added tools like go.sum and tools.go for better tracking and consistency of development tools. Changes are made for improving project setup and future maintenance.
# Conflicts: # go.mod # go.sum
Updated the Go version in GitHub workflows from 1.18 and 1.19 to 1.21 for better compatibility with newer features and performance improvements. The golangci-lint version was also updated for more recent bug-fixes and changes. These updates ensure that the CI environment used for automated testing and linting is in sync with the latest Go language version, hence provides better support for the latest Go language syntax, features, and modules.
Removed Go version 1.20.x from the GitHub workflows' compatibility-test and kept version 1.21.x to ensure our codebase matches the latest Go syntax, features, and modules. This change is aimed at enhancing performance and compatibility.
This commit removes the golangci-lint.yml workflow file as the linting process is now integrated into the test.yml workflow, reducing redundancy and streamlining CI/CD operations. Additionally, a Logger interface was introduced in the server.go file to improve logging practices across the application.
Logger interface has been removed from httpinfra/server.go as part of a refactoring process. This change is made to further separate concerns in the codebase, improving maintainability. Logger interface will be introduced elsewhere as necessary.
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.
Updated Go version and all module dependencies in go.mod for better compatibility and performance. Added an 'init-tools' makefile target to ensure all required tools for development are installed and properly set up. Adjusted 'lint' target to use local golangci-lint. Made minor code style adjustment in httptest.go. Added tools like go.sum and tools.go for better tracking and consistency of development tools. Changes are made for improving project setup and future maintenance.