Skip to content

Conversation

cheenamalhotra
Copy link
Member

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.

@cheenamalhotra cheenamalhotra requested review from paulmedynski and a team January 8, 2025 13:38
@cheenamalhotra
Copy link
Member Author

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@cheenamalhotra cheenamalhotra changed the title Fis synonym count issues on debug Fix synonym count issues on debug Jan 8, 2025
@cheenamalhotra cheenamalhotra merged commit d8d9fc4 into main Jan 8, 2025
128 of 130 checks passed
@cheenamalhotra cheenamalhotra deleted the dev/cheena/fix-synonym-count branch January 8, 2025 15:20
@cheenamalhotra cheenamalhotra linked an issue Jan 8, 2025 that may be closed by this pull request
Copy link

codecov bot commented Jan 8, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 72.76%. Comparing base (7d93424) to head (ee34563).
Report is 2 commits behind head on main.

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     
Flag Coverage Δ
addons 92.58% <ø> (ø)
netcore 75.51% <100.00%> (+0.02%) ⬆️
netfx 71.22% <100.00%> (-0.02%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@mdaigle mdaigle added this to the 7.0-preview1 milestone Jan 14, 2025
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.

Debug Assertion Failure : incorrect initial ParseSynonyms size
4 participants