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

Issue / .NET 8 Update #83

Merged
merged 64 commits into from
Dec 29, 2023
Merged

Issue / .NET 8 Update #83

merged 64 commits into from
Dec 29, 2023

Conversation

SeferMirza
Copy link
Contributor

@SeferMirza SeferMirza commented Dec 4, 2023 •
edited
Loading

Update supported .NET version to 8

Tasks

  • Learn .NET 8 Epic / .NET 8 learn-dotnet#17
  • Research C# features
  • Migrate Plan
  • Update sample project to .NET 8
  • Migration Document
  • Upgrade Issues => No issues 👏
  • Migrations page on website

Additional Tasks

  • migrate current exception handling to use IExceptionHandler
  • A constant message will be given to the message of Internal Server Errors.
  • Detail should be printed to the log.
  • Since there are no nullable prameters in the entities get endpoint in
    do.test.blueprint.application, it does not pass the default check even if
    default is given.
  • Migrate ISystem to use TimeProvider. Use FakeTimeProvider for
    mocking time in tests.
  • use ProblemDetails for exception response.
  • Use useStatusCodePages
  • Add Primary constructor usage for exception in learn dotnet Issue / Primary Constructors For Exceptions learn-dotnet#27

@SeferMirza SeferMirza self-assigned this Dec 4, 2023
cihandeniz
cihandeniz reviewed Dec 26, 2023
View reviewed changes
test/blueprints/Do.Test.Blueprints.Service/Singleton.cs Outdated Show resolved Hide resolved
test/blueprints/Do.Test.Blueprints.Service/Singleton.cs Outdated Show resolved Hide resolved
test/blueprints/Do.Test.Blueprints.Service.Test/DependencyInjection/SystemNow.cs Outdated Show resolved Hide resolved
docs/architecture/feature.md Outdated Show resolved Hide resolved
src/blueprints/Do.Blueprints.Service.Application/Core/Mock/ResettableFakeTimeProvider.cs Outdated Show resolved Hide resolved
unreleased.md Outdated Show resolved Hide resolved
unreleased.md Outdated Show resolved Hide resolved
unreleased.md Outdated Show resolved Hide resolved
....Blueprints.Service.Application/ExceptionHandling/Default/DefaultExceptionHandlingFeature.cs Outdated Show resolved Hide resolved
....Blueprints.Service.Application/ExceptionHandling/Default/DefaultExceptionHandlingFeature.cs Outdated Show resolved Hide resolved
@SeferMirza SeferMirza marked this pull request as ready for review December 28, 2023 13:50
@SeferMirza SeferMirza merged commit 725eec7 into mouseless:main Dec 29, 2023
8 checks passed
@SeferMirza SeferMirza deleted the issue/net-8-update branch December 29, 2023 15:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@alptuncs alptuncs alptuncs left review comments

@cihandeniz cihandeniz cihandeniz approved these changes

Assignees

@SeferMirza SeferMirza

@alptuncs alptuncs

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@SeferMirza @cihandeniz @alptuncs