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.

Sign up for GitHub

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

Jump to bottom

SSA and generics (go1.18) #11

Closed
Tracked by #2649
ldez opened this issue Mar 24, 2022 · 4 comments
Closed
Tracked by #2649

SSA and generics (go1.18) #11

ldez opened this issue Mar 24, 2022 · 4 comments

Comments

@ldez
Copy link
Contributor

ldez commented Mar 24, 2022 •
edited
Loading

Currently, SSA is not working with generics.

So your linter produces a panic when it is used with generics.

There is an issue open about that in the Go repository: golang/go#48525

Inside golangci-lint, we disabled your linters: golangci/golangci-lint#2649

You have 2 solutions:

  • waiting for a version of SSA that will support generics
  • dropping the SSA analyzers and using something else to analyze the code.

Related to golang/go#50558
@ldez ldez mentioned this issue Mar 24, 2022
Closed
13 tasks
@sonatard
Copy link
Owner

Thanks for reporting issue!

I wait for Go team to fix golang/go#48525.

@ldez
Copy link
Contributor Author

ldez commented Jul 19, 2022

SSA has been updated and it seems to work inside golangci-lint.

@ldez ldez closed this as completed Jul 19, 2022
@ldez
Copy link
Contributor Author

ldez commented Jul 19, 2022

in fact, the problem is still here

@ldez ldez reopened this Jul 19, 2022
@ldez
Copy link
Contributor Author

ldez commented Jul 19, 2022

I did some more extensive testing and the issue seems to be resolved.

@ldez ldez closed this as completed Jul 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@sonatard @ldez