Skip to content

Organizes SPA source files in the Visual Studio project tree#17

Open
jSylvestre wants to merge 1 commit intomainfrom
JCS/ShowInVsSolution
Open

Organizes SPA source files in the Visual Studio project tree#17
jSylvestre wants to merge 1 commit intomainfrom
JCS/ShowInVsSolution

Conversation

@jSylvestre
Copy link
Member

@jSylvestre jSylvestre commented Mar 23, 2026

Sets LinkBase="client" so that the frontend source files are grouped under a virtual "client" folder in the Solution Explorer instead of appearing at the project root.

Summary by CodeRabbit

  • Chores
    • Updated project build configuration to optimize how frontend files are organized and displayed in the development environment with no impact on application functionality.

Sets LinkBase="client" so that the frontend source files are grouped under a virtual "client" folder in the Solution Explorer instead of appearing at the project root.
@jSylvestre jSylvestre requested a review from sprucely March 23, 2026 14:00
@coderabbitai
Copy link

coderabbitai bot commented Mar 23, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 37757063-af77-47a5-bbdb-7155cc58b918

📥 Commits

Reviewing files that changed from the base of the PR and between d58444f and 6644fc0.

📒 Files selected for processing (1)
  • server/server.csproj

📝 Walkthrough

Walkthrough

Updated the MSBuild configuration in server/server.csproj to add LinkBase="client" metadata to the SPA frontend file inclusion rule. This modifies how included files from the SPA root are linked in the project structure without changing which files are included or excluded.

Changes

Cohort / File(s) Summary
MSBuild Configuration
server/server.csproj
Added LinkBase="client" metadata to the <None Include> entry for SPA frontend files ($(SpaRoot)\**), changing how these files are organized/displayed in Visual Studio's project structure.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A single line, so small and neat,
LinkBase="client" makes the path complete!
Visual Studio's structure now shines bright,
SPA files nestled, perfectly right! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: organizing SPA source files in the Visual Studio project tree by adding LinkBase="client" to group them under a virtual folder.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch JCS/ShowInVsSolution

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@jSylvestre jSylvestre requested a review from srkirkland March 23, 2026 14:00
@jSylvestre
Copy link
Member Author

Client folder isn't visible in VS without this.

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.

1 participant