Skip to content
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

Fix misleading version input description #33

Merged
merged 6 commits into from Jul 6, 2020

Conversation

sayboras
Copy link
Member

@sayboras sayboras commented Jul 5, 2020

Fixes #32

Signed-off-by: Tam Mach sayboras@yahoo.com

Signed-off-by: Tam Mach <sayboras@yahoo.com>
@sayboras sayboras requested a review from a team July 5, 2020 01:09
Copy link

@iwankgb iwankgb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't it use v1.28.0 (or 1.28.1 that we should release with fix to header linter)?

@sayboras
Copy link
Member Author

sayboras commented Jul 5, 2020

Shouldn't it use v1.28.0 (or 1.28.1 that we should release with fix to header linter)?

I think golangci-lint will refer to https://github.com/golangci/golangci-lint/blob/master/assets/github-action-config.json for the exact version to install.

The reason I didn't update to v1.28 to avoid chicken and egg issue, as golangci-lint uses golangci-lint-action and via versa. Moreover, this PR got recreated before golangci/golangci-lint#1207 is merged.

@iwankgb
Copy link

iwankgb commented Jul 5, 2020

1.28.0 seems to be available now in all necessary places so I think it's safe to upgrade version in this PR too.

@sayboras sayboras requested a review from iwankgb July 5, 2020 23:10
@iwankgb iwankgb merged commit d737e6d into golangci:master Jul 6, 2020
@sayboras sayboras deleted the bugfix/update-description branch July 6, 2020 04:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Input description for version is misleading
2 participants