Skip to content
This repository has been archived by the owner on Apr 9, 2024. It is now read-only.

fix: Update to GO 1.18 so scan works on new projects #75

Merged

Conversation

adadkins
Copy link
Contributor

If you run gokart scan on go 1.18 projects, you get an error:

internal error: package "time" without types was imported from "github.com/Omaze/cl-sweeps-api/internal/common"

We need to have a 1.18 build for gokart for it to work with new go projects

@ynsta
Copy link

ynsta commented Sep 2, 2022

Would also need this fix.

@adadkins
Copy link
Contributor Author

adadkins commented Sep 7, 2022

@isp1r0 @praetorian-harry Hey sorry to ping, just curious if we can merge this so gokart works with new go projects

@sc-atompower
Copy link

@ynsta @adadkins the go team recently announced https://go.dev/blog/vuln

@adadkins
Copy link
Contributor Author

adadkins commented Sep 8, 2022

@sc-atompower thanks for that, i did not see that announcement. I'm gong to close this PR and switch to using that. Thanks

@adadkins adadkins closed this Sep 8, 2022
@adadkins adadkins deleted the update-to-go1.18 branch September 8, 2022 21:02
@nekochanell
Copy link

@ynsta @adadkins the go team recently announced https://go.dev/blog/vuln

It's not even the same thing. GoKart is a static code analyser. GoVulnCheck is for dependency vulnerability management.

This issue should be reopened.

@adadkins adadkins restored the update-to-go1.18 branch September 16, 2022 21:25
@adadkins
Copy link
Contributor Author

Reopening

@adadkins adadkins reopened this Sep 16, 2022
Copy link
Contributor

@praetorian-thendrickson praetorian-thendrickson left a comment

Choose a reason for hiding this comment

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

Thanks!

@praetorian-thendrickson
Copy link
Contributor

Hey @adadkins thank you for your help with this PR. I updated it slightly, so the current main branch #80 should now work.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants