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.

Sign up for GitHub

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

Jump to bottom

Update to .NET 8 #412

Merged
merged 9 commits into from
Nov 14, 2023
Merged

Update to .NET 8 #412

merged 9 commits into from
Nov 14, 2023

Conversation

martincostello
Copy link
Owner

  • Update to .NET 8.
  • Fix new code analysis warnings.

@martincostello martincostello added enhancement New feature or request dependencies Pull requests that update a dependency file .NET Pull requests that update .net code labels May 4, 2023
@martincostello martincostello mentioned this pull request May 4, 2023
Closed
@codecov
Copy link

codecov bot commented May 4, 2023 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (9cb738f) 98.23% compared to head (c162328) 98.19%.
Report is 1 commits behind head on main.

Additional details and impacted files 
@@            Coverage Diff             @@
##             main     #412      +/-   ##
==========================================
- Coverage   98.23%   98.19%   -0.05%     
==========================================
  Files          15       15              
  Lines         340      332       -8     
  Branches       96       94       -2     
==========================================
- Hits          334      326       -8     
  Misses          3        3              
  Partials        3        3
Flag Coverage Δ
linux ?
macos 98.19% <100.00%> (-0.05%) ⬇️
windows ?

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.

martincostello added a commit that referenced this pull request May 18, 2023
@martincostello
.NET 8 preparation
b759a88 
Cherry-pick various changes from #412.
@martincostello martincostello mentioned this pull request May 18, 2023
Merged
martincostello added a commit that referenced this pull request May 18, 2023
@martincostello
.NET 8 preparation
b06632c 
Cherry-pick various changes from #412.
@martincostello martincostello force-pushed the dotnet-vnext branch 2 times, most recently from 8d11f17 to 6920130 Compare May 22, 2023 13:19
martincostello added a commit that referenced this pull request Jun 16, 2023
@martincostello
Suppress CA1861
358e334 
Cherry-pick change from #412 to suppress CA1861 in tests.
@martincostello martincostello mentioned this pull request Jun 16, 2023
Merged
martincostello added a commit that referenced this pull request Jun 16, 2023
@martincostello
Suppress CA1861
630dc8d 
Cherry-pick change from #412 to suppress CA1861 in tests.
@martincostello martincostello force-pushed the dotnet-vnext branch 3 times, most recently from d355e10 to 4144757 Compare June 23, 2023 10:49
@martincostello martincostello force-pushed the dotnet-vnext branch 2 times, most recently from b30f331 to 365c7d4 Compare July 13, 2023 15:08
@martincostello martincostello force-pushed the dotnet-vnext branch 5 times, most recently from 35adf3a to 15c9292 Compare August 10, 2023 11:48
@martincostello martincostello force-pushed the dotnet-vnext branch 2 times, most recently from eb3126c to c077298 Compare August 30, 2023 14:13
@martincostello martincostello force-pushed the dotnet-vnext branch 2 times, most recently from b445156 to 7b5629f Compare September 12, 2023 17:23
@martincostello martincostello force-pushed the dotnet-vnext branch 2 times, most recently from c2f29a4 to a553fdb Compare October 25, 2023 18:23
martincostello and others added 8 commits November 9, 2023 07:06
@martincostello
Update to .NET 8 preview 7
a067406 
- Update to preview 7 of .NET 8.
- Use `UseArtifactsOutput=true` instead of explicitly setting the output path, as well as dropping explicit use of `--configuration` now that the default is `Release`.
- Remove Microsoft.SourceLink.GitHub as it is now included in the .NET SDK.
@costellobot @martincostello
Update .NET SDK (#492)
7a1913d 
Update .NET SDK to version 8.0.100-rc.1.23455.8.

---
updated-dependencies:
- dependency-name: Microsoft.NET.Sdk
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: costellobot <[email protected]>
@martincostello
Update NuGet package
62ae3bc 
Update NuGet package for RC.1.
@costellobot @martincostello
Update .NET SDK (#497)
9df9fa6 
Update .NET SDK to version 8.0.100-rc.1.23463.5.

---
updated-dependencies:
- dependency-name: Microsoft.NET.Sdk
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: costellobot <[email protected]>
@martincostello
Use C# 12 features
8f0b5eb 
- Use primary constructors where relevant.
- Use collection literals where relevant.
- Apply some IDE suggestions.
- Remove redundant suppressions.
- Remove `CommonAssemblyInfo`.
@martincostello
Use C# 12 in README
8357bac 
Use C#12 syntax in the README sample.
@costellobot @martincostello
Update .NET SDK to 8.0.100-rc.2.23502.2 (#505)
af86c99 
* Update .NET SDK

Update .NET SDK to version 8.0.100-rc.2.23502.2.

---
updated-dependencies:
- dependency-name: Microsoft.NET.Sdk
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: costellobot <[email protected]>

* Bump Microsoft.AspNetCore.Mvc.Testing from 8.0.0-rc.1.23421.29 to 8.0.0-rc.2.23480.2

Bumps Microsoft.AspNetCore.Mvc.Testing from 8.0.0-rc.1.23421.29 to 8.0.0-rc.2.23480.2.

---
updated-dependencies:
- dependency-name: Microsoft.AspNetCore.Mvc.Testing
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: costellobot <[email protected]>

---------

Signed-off-by: costellobot <[email protected]>
@martincostello
Use terminal logger
529b9ad 
Use the new MSBuild terminal logger.
@costellobot
Update .NET SDK to 8.0.100 (#524)
c162328 
* Update .NET SDK

Update .NET SDK to version 8.0.100.

---
updated-dependencies:
- dependency-name: Microsoft.NET.Sdk
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: costellobot <[email protected]>

* Bump Microsoft.AspNetCore.Mvc.Testing from 8.0.0-rc.2.23480.2 to 8.0.0

Bumps Microsoft.AspNetCore.Mvc.Testing from 8.0.0-rc.2.23480.2 to 8.0.0.

---
updated-dependencies:
- dependency-name: Microsoft.AspNetCore.Mvc.Testing
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: costellobot <[email protected]>

---------

Signed-off-by: costellobot <[email protected]>
@martincostello martincostello marked this pull request as ready for review November 14, 2023 17:50
@martincostello martincostello merged commit 53f94ee into main Nov 14, 2023
12 checks passed
@martincostello martincostello deleted the dotnet-vnext branch November 14, 2023 17:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
dependencies Pull requests that update a dependency file enhancement New feature or request .NET Pull requests that update .net code
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@martincostello @costellobot