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

[CI] Move to latest version of golangci-lint #5993

Closed
nishantwrp opened this issue Aug 7, 2022 · 14 comments · Fixed by #6567
Closed

[CI] Move to latest version of golangci-lint #5993

nishantwrp opened this issue Aug 7, 2022 · 14 comments · Fixed by #6567
Assignees
Labels
area/ci Continuous Integration issue/willfix This issue will be worked on

Comments

@nishantwrp
Copy link
Member

Move to the latest version of golangci-lint after golangci/golangci-lint-action#535 is fixed. #5991 rollbacks the version of golangci-lint due to the given bug.


Contributor Guides and Handbook

@nishantwrp nishantwrp added the area/ci Continuous Integration label Aug 7, 2022
@alphaX86
Copy link
Member

@nishantwrp is this completed? If so, can we safely close this issue?

@nishantwrp
Copy link
Member Author

Hi @alphaX86, no it's not. We need to start using latest version of golangci-lint again, which will only be possible after golangci/golangci-lint-action#535 is fixed.

@alphaX86
Copy link
Member

Ok understood. Thanks for clearing this

@stale
Copy link

stale bot commented Sep 22, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the issue/stale Issue has not had any activity for an extended period of time label Sep 22, 2022
@nishantwrp nishantwrp added the issue/willfix This issue will be worked on label Sep 22, 2022
@deepeshaburse deepeshaburse added issue/remind Issue progress check and removed issue/stale Issue has not had any activity for an extended period of time labels Sep 25, 2022
@github-actions
Copy link

Checking in... it has been awhile since we've heard from you on this issue. Are you still working on it? Please let us know and please don't hesitate to contact a MeshMate or any other community member for assistance.


        Be sure to join the community, if you haven't yet and please leave a ⭐ star on the project 😄

@nishantwrp
Copy link
Member Author

golangci/golangci-lint-action#535 is still not fixed. We can only work on this after the issue in golangci-lint gets fixed.

@nishantwrp nishantwrp removed the issue/remind Issue progress check label Sep 28, 2022
@acald-creator
Copy link
Contributor

I will be working optimizing the GitHub Actions workflows.

There are several different workflows, and one of the suggestion is to create a separate workflow specifically for golangci-lint so that when it comes to reviewing PRs, we can work faster on the PR specific to ui or anything else that is not Go.

@leecalcote
Copy link
Member

Hopefully, we'll get a bird's eye view assembled in the Meshery Test Plan spreadsheet with an overview of the workflows included.

@segunjkf
Copy link

can i work on this?

@leecalcote
Copy link
Member

@segunjkf that'd be great, yes.

@segunjkf
Copy link

@leecalcote Golangci-lint, in my opinion, should have its own link workflow. I looked around and discovered it in the build-ui-and-server.yaml workflow.

@nishantwrp
Copy link
Member Author

Hi, I'm not sure if this issue can be fixed until golangci/golangci-lint-action#535 is fixed.

@acald-creator
Copy link
Contributor

I already have a branch working on the golangci-lint GitHub Actions workflow as well as trying out the different linting settings. I will submit a PR today.

As for the open issue, I haven't encountered this with Go 1.19, since we're not using Go 1.18

@acald-creator
Copy link
Contributor

For your reference, here is my branch in progress. The only reason why I haven't submitted the PR yet because of the linting settings which will cause the workflow to instantly fail.

master...acald-creator:meshery:enhance-golangci-lint

I am also working on a separate branch as well with just the workflow.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ci Continuous Integration issue/willfix This issue will be worked on
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants