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

gopbuild: RegisterPackagePatch #275

Merged
merged 2 commits into from
Aug 28, 2024

context: export SourcePackage

06080f8
Select commit
Loading
Failed to load commit list.
Merged

gopbuild: RegisterPackagePatch #275

context: export SourcePackage
06080f8
Select commit
Loading
Failed to load commit list.
qiniu-x / golangci-lint failed Aug 28, 2024 in 35s

golangci-lint found 5 issues related to your changes

If you have any questions about this comment, feel free to raise an issue here.

Annotations

Check warning on line 679 in context.go

See this annotation in the file changed.

@qiniu-x qiniu-x / golangci-lint

context.go#L679

cognitive complexity 50 of func `(*Context).buildPackage` is high (> 30) (gocognit)

Check warning on line 354 in context.go

See this annotation in the file changed.

@qiniu-x qiniu-x / golangci-lint

context.go#L354

string `main` has 5 occurrences, make it a constant (goconst)

Check warning on line 122 in context.go

See this annotation in the file changed.

@qiniu-x qiniu-x / golangci-lint

context.go#L122

`if sp.Info == nil` has complex nested blocks (complexity: 14) (nestif)

Check warning on line 241 in gopbuild/build.go

See this annotation in the file changed.

@qiniu-x qiniu-x / golangci-lint

gopbuild/build.go#L241

typeSwitchVar: 1 case can benefit from type switch with assignment (gocritic)

Check warning on line 231 in gopbuild/build.go

See this annotation in the file changed.

@qiniu-x qiniu-x / golangci-lint

gopbuild/build.go#L231

`if gop` has complex nested blocks (complexity: 5) (nestif)