Skip to content

feat: update test projects to dotnet 8 - #239

Merged
wmundev merged 2 commits into
twcclegg:mainfrom
wmundev:feature/upgrade-test-project-to-dotnet8
Mar 31, 2024
Merged

feat: update test projects to dotnet 8#239
wmundev merged 2 commits into
twcclegg:mainfrom
wmundev:feature/upgrade-test-project-to-dotnet8

Conversation

@wmundev

@wmundev wmundev commented Mar 25, 2024

Copy link
Copy Markdown
Collaborator

No description provided.

@wmundev wmundev self-assigned this Mar 25, 2024
@codecov

codecov Bot commented Mar 25, 2024

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 83.42%. Comparing base (85d1efe) to head (36a5567).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #239      +/-   ##
==========================================
+ Coverage   83.39%   83.42%   +0.02%     
==========================================
  Files          30       30              
  Lines        3499     3499              
  Branches      978      978              
==========================================
+ Hits         2918     2919       +1     
  Misses        280      280              
+ Partials      301      300       -1     

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

@twcclegg

Copy link
Copy Markdown
Owner

We can add net8 whenever, but we want to keep net7 until at least May 14.

@wmundev

wmundev commented Mar 25, 2024

Copy link
Copy Markdown
Collaborator Author

thanks Thomas, i'll restore dotnet 7

@wmundev
wmundev marked this pull request as ready for review March 31, 2024 04:09
@wmundev
wmundev merged commit 0d2a146 into twcclegg:main Mar 31, 2024
twcclegg added a commit that referenced this pull request Aug 26, 2026
int.GetHashCode() returns the int itself, so XOR-ing Start directly
is equivalent. Addresses CodeQL cs/useless-gethashcode-call (alert #239).

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
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.

2 participants