-
Couldn't load subscription status.
- Fork 316
Fix synonym count issues on debug #3098
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
Conversation
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #3098 +/- ##
==========================================
- Coverage 72.76% 72.76% -0.01%
==========================================
Files 283 283
Lines 58968 58996 +28
==========================================
+ Hits 42908 42926 +18
- Misses 16060 16070 +10
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Addresses #3016 due to deprecated keywords count mismatch..
This change is more effective for unit testing than #3088, hence proposing the same.
The deprecated keywords weren't tracked before, and this PR removes the counter. If keywords are needed to be deprecated, we can deprecate them on docs and follow deprecation path in future releases. We need not manage counters to be modified as they are very sensitive to changes and affect unit testing in debug mode.