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

Remove datatables.net compatible code. #22159

Merged
merged 1 commit into from
Feb 15, 2025
Merged

Remove datatables.net compatible code. #22159

merged 1 commit into from
Feb 15, 2025

Conversation

maliming
Copy link
Member

@maliming maliming commented Feb 15, 2025

Related #22138

@maliming maliming added this to the 9.0-patch-final milestone Feb 15, 2025
Copy link

codecov bot commented Feb 15, 2025

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Project coverage is 51.85%. Comparing base (2fb1d15) to head (9bb9573).
Report is 35 commits behind head on rel-9.0.

Files with missing lines Patch % Lines
...es/DatatablesNet/DatatablesNetScriptContributor.cs 0.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           rel-9.0   #22159      +/-   ##
===========================================
- Coverage    51.85%   51.85%   -0.01%     
===========================================
  Files         3174     3175       +1     
  Lines       101816   101831      +15     
  Branches      7710     7711       +1     
===========================================
+ Hits         52797    52804       +7     
- Misses       47401    47408       +7     
- Partials      1618     1619       +1     

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

@berkansasmaz berkansasmaz merged commit b382847 into rel-9.0 Feb 15, 2025
3 of 5 checks passed
@berkansasmaz berkansasmaz deleted the dataTables branch February 15, 2025 18:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants