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

Update Linters #247

Merged
merged 7 commits into from
Sep 2, 2022
Merged

Update Linters #247

merged 7 commits into from
Sep 2, 2022

Conversation

lowlydba
Copy link
Owner

@lowlydba lowlydba commented Sep 1, 2022

Tweak/update linters

@github-actions
Copy link

github-actions bot commented Sep 1, 2022

✅ TSQLLint Summary

No issues found.

Detailed results.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Sep 1, 2022

badge

Code Coverage Summary

Filename                    Stmts    Miss  Cover    Missing
------------------------  -------  ------  -------  -----------------------------------------------------------------------
[dbo].[sp_doc]                 91       3  96.70%   93-94, 391
[dbo].[sp_estindex]           207       5  97.58%   186, 739-744
[dbo].[sp_helpme]              86       1  98.84%   337
[dbo].[sp_sizeoptimiser]      240      18  92.50%   145, 178, 226, 241, 661, 871, 926, 930, 934, 938, 1078, 1139, 1221-1226
TOTAL                         624      27  95.67%

Results for commit: fa09497

Minimum allowed coverage is 90%

♻️ This comment has been updated with latest results

@lowlydba lowlydba changed the title test Update Linters Sep 2, 2022
@lowlydba lowlydba merged commit b47d1e1 into main Sep 2, 2022
@lowlydba lowlydba deleted the tsqllint-updates branch September 2, 2022 00:47
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

Successfully merging this pull request may close these issues.

None yet

1 participant