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

Sqlcheck does not show line number on which hint or antipattern is reported #24

Closed
KetkiT opened this issue Jul 17, 2018 · 0 comments
Closed
Assignees
Milestone

Comments

@KetkiT
Copy link

KetkiT commented Jul 17, 2018

Sqlcheck does not show line number on which antipattern or hint is reported . Currently it gives following format

[D:\temp_func.sql]: (LOW RISK) (QUERY ANTI-PATTERN) Spaghetti Query Alert
[Matching Expression: ]

[D:\temp_func.sql]: (LOW RISK) (QUERY ANTI-PATTERN) UNION Usage
[Matching Expression: union]

can it give line no as well ?
e.g . [D:\temp_func.sql]: (LOW RISK) (QUERY ANTI-PATTERN) UNION Usage
[Matching Expression: union] [34]

This becomes very useful when it is a huge file. With line number (starting position of the query) you can directly navigate to the specific query.

Line number are very common in all code checker , as they give better user experience .

Should line number be added here?

Thanks

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

No branches or pull requests

2 participants