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

MudTable: Add nullable annotation. #8926

Merged
merged 1 commit into from
May 8, 2024

Conversation

ScarletKuro
Copy link
Member

@ScarletKuro ScarletKuro commented May 8, 2024

Description

Part of this issue #6535
Fixes: #7629

NB! There probably will be adjustment in future for the annotation. Currently some null checks can be excessive in places where I wasn't sure.

How Has This Been Tested?

No tests required.

Type of Changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change, improve null diagnostic warnings)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation (fix or improvement to the website or code docs)

Checklist

  • The PR is submitted to the correct branch (dev).
  • My code follows the code style of this project.
  • I've added relevant tests.

Copy link

codecov bot commented May 8, 2024

Codecov Report

Attention: Patch coverage is 70.40359% with 66 lines in your changes are missing coverage. Please review.

Project coverage is 90.12%. Comparing base (28bc599) to head (e8fd3d8).
Report is 170 commits behind head on dev.

Files Patch % Lines
src/MudBlazor/Components/Table/MudTr.razor.cs 51.42% 0 Missing and 17 partials ⚠️
src/MudBlazor/Components/Table/MudTable.razor.cs 62.50% 3 Missing and 6 partials ⚠️
...Blazor/Components/Table/MudTableSortLabel.razor.cs 66.66% 2 Missing and 5 partials ⚠️
src/MudBlazor/Components/Table/TableContext.cs 72.72% 2 Missing and 4 partials ⚠️
.../MudBlazor/Components/Table/MudTableGroupRow.razor 0.00% 0 Missing and 4 partials ⚠️
...MudBlazor/Components/Table/MudTemplateColumn.razor 0.00% 0 Missing and 4 partials ⚠️
...dBlazor/Components/Table/MudTableGroupRow.razor.cs 81.25% 0 Missing and 3 partials ⚠️
.../MudBlazor/Components/Table/MudTablePager.razor.cs 88.23% 0 Missing and 2 partials ⚠️
src/MudBlazor/Components/Table/MudTr.razor 0.00% 0 Missing and 2 partials ⚠️
...Blazor/Extensions/TableButtonPositionExtensions.cs 77.77% 0 Missing and 2 partials ⚠️
... and 10 more
Additional details and impacted files
@@            Coverage Diff             @@
##              dev    #8926      +/-   ##
==========================================
+ Coverage   89.82%   90.12%   +0.29%     
==========================================
  Files         412      426      +14     
  Lines       11878    12310     +432     
  Branches     2364     2422      +58     
==========================================
+ Hits        10670    11094     +424     
+ Misses        681      669      -12     
- Partials      527      547      +20     

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

@ScarletKuro ScarletKuro added enhancement New feature or request and removed PR: needs review labels May 8, 2024
@ScarletKuro ScarletKuro merged commit 655d2e0 into MudBlazor:dev May 8, 2024
4 of 5 checks passed
@ScarletKuro ScarletKuro deleted the table_nullable branch May 8, 2024 20:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

TableState.SortLabel initialized to null
1 participant