Skip to content

Add execinquery linter #2677

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

Merged
merged 12 commits into from
Mar 29, 2022
Merged

Add execinquery linter #2677

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
b2bca04
Add execinquery
1uf3 Mar 22, 2022
c37041f
Add more test case and change version
1uf3 Mar 23, 2022
eba848c
review: fix go mod
ldez Mar 29, 2022
2cb86de
review: preset and alphabetical order
ldez Mar 29, 2022
23575d9
review: rename files and fix tests
ldez Mar 29, 2022
73ef64f
review: fix analyzer
ldez Mar 29, 2022
eef9f55
review: fix tests
ldez Mar 29, 2022
4f0673a
review: fix imports
ldez Mar 29, 2022
86ad25f
update go.mod
1uf3 Mar 29, 2022
7e4a5d4
delete previous version
1uf3 Mar 29, 2022
5bcdbea
review: fix go.mod
ldez Mar 29, 2022
f71c595
review: remove temp fix
ldez Mar 29, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
review: remove temp fix
  • Loading branch information
@ldez
ldez committed Mar 29, 2022
commit f71c59590285b30393e5d163b5acfb27946cfe29
5 changes: 1 addition & 4 deletions pkg/golinters/execinquery.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,13 +8,10 @@ import (
)

func NewExecInQuery() *goanalysis.Linter {
const linterName = "execinquery"

a := execinquery.Analyzer
a.Name = linterName // TODO the name must change inside the linter.

return goanalysis.NewLinter(
linterName,
a.Name,
a.Doc,
[]*analysis.Analyzer{a},
nil,
Expand Down