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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Suggestion fixes #244

Merged
merged 6 commits into from
Aug 19, 2022
Merged

Suggestion fixes #244

merged 6 commits into from
Aug 19, 2022

Conversation

lowlydba
Copy link
Owner

Description

Don't trigger a line item for timestamp columns named timestamp 馃う馃徎

Issue: Fixes

How Has This Been Tested?

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)

Checklist:

  • I have updated the Version number of any scripts modified.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@github-actions
Copy link

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: ab70f47

Minimum allowed coverage is 90%

鈾伙笍 This comment has been updated with latest results

@lowlydba lowlydba merged commit 443f808 into main Aug 19, 2022
@lowlydba lowlydba deleted the suggestion-fixes branch August 19, 2022 00:48
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