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

golangci-lint 1.51.1 #122351

Closed
wants to merge 1 commit into from
Closed

Conversation

andig
Copy link
Contributor

@andig andig commented Feb 5, 2023

Created with brew bump-formula-pr.

Note: according to golangci/golangci-lint#3565, golangci-lint 1.51.0 and this PR are currently broken when built on Go 1.19 and used with Go 1.20.

This PR hence depends on #122082.

@BrewTestBot BrewTestBot added go Go use is a significant feature of the PR or issue bump-formula-pr PR was created using `brew bump-formula-pr` labels Feb 5, 2023
@andig andig closed this Feb 5, 2023
@andig andig deleted the bump-golangci-lint-1.51.1 branch February 5, 2023 14:20
@andig andig restored the bump-golangci-lint-1.51.1 branch February 5, 2023 14:20
@andig andig reopened this Feb 5, 2023
@andig andig mentioned this pull request Feb 6, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Feb 9, 2023

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. To keep this pull request open, add a help wanted or in progress label.

@github-actions github-actions bot added the stale No recent activity label Feb 9, 2023
@github-actions github-actions bot closed this Feb 10, 2023
@chenrui333
Copy link
Member

chenrui333 commented Feb 11, 2023

NoMethodError: undefined method rfc3339' for 2023-02-05 00:48:10 UTC:Time`

The issue is resolved now.

relates to #122483

@chenrui333 chenrui333 reopened this Feb 11, 2023
@github-actions github-actions bot removed the stale No recent activity label Feb 11, 2023
@chenrui333 chenrui333 force-pushed the bump-golangci-lint-1.51.1 branch from dd76f99 to 2faa0e2 Compare February 11, 2023 09:46
@chenrui333 chenrui333 added the CI-requeued PR has been re-added to the queue label Feb 11, 2023
@andig
Copy link
Contributor Author

andig commented Feb 11, 2023

We should wait for Go 1.20 to make sure this will work.

@BrewTestBot
Copy link
Member

🤖 A scheduled task has triggered a merge.

@andig andig deleted the bump-golangci-lint-1.51.1 branch February 11, 2023 11:26
@chenrui333
Copy link
Member

We should wait for Go 1.20 to make sure this will work.

The formula would get revision bump to regression test against go1.20, so it should be fine.

@chenrui333
Copy link
Member

I am referring to this commit

@github-actions github-actions bot added the outdated PR was locked due to age label Mar 15, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 15, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bump-formula-pr PR was created using `brew bump-formula-pr` CI-requeued PR has been re-added to the queue go Go use is a significant feature of the PR or issue outdated PR was locked due to age
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants