-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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 1.19 to go_version matrix to run tests against latest go versions. #1252
Conversation
Did You run code gofmt on 1.19? I think it's required by go 1.19, there was some changes in comments formatting |
.github/workflows/main.yml
Outdated
@@ -6,13 +6,14 @@ jobs: | |||
runs-on: ubuntu-latest | |||
strategy: | |||
matrix: | |||
go_version: ["1.18.1", "1.17.6", "1.16.5"] | |||
go_version: ["^1.18", "^1.17", "^1.16"] |
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.
the ^
symbol is not required, the 1.18
will work as expected
@kubaceg Yes. The problem is correct gofmt format is different for versions 1.19 and prior 1.19
commit changes:
Then I run
Number shows count of different lines - indeed Than I format code with go 1.19:
and check with the same versions again:
So if strict format enabled there's no version suitable for |
Outdated. |
Summary
Test testify against actual golang version.
Changes
Motivation
To ensure testify is ready for actuality and future.
Related issues