NET-10: Set up .NET Core 6 project scaffolding#18
Open
devin-ai-integration[bot] wants to merge 1 commit intomasterfrom
Open
NET-10: Set up .NET Core 6 project scaffolding#18devin-ai-integration[bot] wants to merge 1 commit intomasterfrom
devin-ai-integration[bot] wants to merge 1 commit intomasterfrom
Conversation
- Add SampleWebApp.Core.sln with net6.0 projects: - src/DataLayer (classlib, EF Core 6 + SqlServer provider) - src/BizLayer (classlib) - src/ServiceLayer (classlib) - src/SampleWebApp (ASP.NET Core MVC) - tests/Tests (xUnit) - Add SampleWebAppDbContext (EF Core) and per-layer DI extension methods (AddDataLayer / AddServiceLayer / AddBizLayer) wired up in Program.cs - Add appsettings.json plus Development/Staging/Production overrides and the SampleWebAppDb connection string - Legacy EF6/MVC5 projects at the repo root are left untouched; the new .NET Core 6 tree lives alongside them for incremental migration Co-Authored-By: unknown <>
Author
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Establishes the foundational .NET Core 6 project structure for the migration tracked in NET-10. The legacy ASP.NET MVC5 / EF6 projects at the repo root are left untouched; the new .NET Core 6 tree lives alongside them so subsequent phases (starting with HomeController migration) can move code over incrementally.
New top-level layout:
Key pieces wired up:
DataLayer/DataClasses/SampleWebAppDbContext.cs— empty EF CoreDbContext(Blogs/Posts/Tags DbSets are deliberately deferred to later phases) with aConnectionStringName = "SampleWebAppDb"constant that matches the legacy Web.config name.AddDataLayer,AddServiceLayer,AddBizLayer) registered insrc/SampleWebApp/Program.cs.AddDataLayercallsUseSqlServerwith theSampleWebAppDbconnection string and throws if it's missing.appsettings.json+appsettings.Development.json/appsettings.Staging.json/appsettings.Production.jsonwith environment-appropriate log levels. Dev/base include the(localdb)\mssqllocaldbconnection string (matches the legacy default).dotnet new mvc— included so the solution builds and a Home page serves out-of-the-box.dotnet build SampleWebApp.Core.slnsucceeds locally with 0 warnings / 0 errors.dotnet testruns but reports "No test is available" (the xUnit project is intentionally empty — actual tests come with later phases).Review & Testing Checklist for Human
src/+tests/folder layout alongside the legacy projects is the intended structure (vs. e.g. replacing the legacy tree or using a different naming scheme like.Coresuffixes).SampleWebAppDbContexthas noDbSets andAddServiceLayer/AddBizLayerare empty no-op registrations. This is intentional for foundation-only, but verify it matches the team's expectation for NET-10 vs. what belongs in later tickets.(localdb)\mssqllocaldbwhich only works on Windows. If devs run on Linux/macOS or CI, a different default (or user-secrets / env override) may be needed.dotnet build SampleWebApp.Core.slnanddotnet run --project src/SampleWebAppon a Windows machine with LocalDB to confirm the app starts and the Home page serves.Notes
SampleWebApp.slnand all existing.csproj/packages.configfiles are unchanged.DataLayer.csprojdeliberately does not defineInternalsVisibleTo("Tests")like the legacy project did; that can be re-added when real entities start moving over.src/SampleWebApp/wwwroot/lib/(bootstrap, jquery, jquery-validation) generated bydotnet new mvc— not hand-written.Link to Devin session: https://app.devin.ai/sessions/c31603dfc46e406fa99067729d9a7851