Skip to content

chore: refactored samples folder and converted solution to using .slnx#75

Merged
gragra33 merged 6 commits into
developfrom
feature/migrate-to-slnx
Apr 27, 2026
Merged

chore: refactored samples folder and converted solution to using .slnx#75
gragra33 merged 6 commits into
developfrom
feature/migrate-to-slnx

Conversation

@mishael-o

@mishael-o mishael-o commented Apr 19, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Switch the repo to src/Blazing.Mvvm.slnx as the main solution entry point and update CI/release/test scripts for the new layout.
  • Reorganize projects under src/Libraries and src/Tests, and move the sample apps from src/samples to repo-root samples with shared sample build/package props and per-sample .slnx files.
  • Add the BLAZMVVM0021 EventCallback two-way binding analyzer, including its code fix, analyzer docs/release notes, sample coverage, and analyzer tests.
  • Fix the README.md filename casing for Linux CI and clean up analyzer/docs/sample markdown links and related documentation.

Resolves #38

mishael-o and others added 3 commits April 19, 2026 15:59
…yzer

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings April 19, 2026 14:58

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

This PR restructures the repository’s sample projects (moving them to a top-level samples/ folder) and standardizes solution entry points by migrating from legacy .sln/.slnf usage to .slnx, while centralizing sample build/package configuration under samples/Directory.* props.

Changes:

  • Updated sample project references to point at src/Libraries/* after moving samples to repo-root samples/.
  • Added samples/Directory.Build.props, samples/Directory.Packages.props, and a local override example to centralize build/package settings for samples.
  • Introduced per-sample .slnx solution entry points and updated CI/release workflows/scripts to build via .slnx.

Reviewed changes

Copilot reviewed 29 out of 779 changed files in this pull request and generated 7 comments.

Show a summary per file
File Description
samples/HybridSamples/HybridSample.MAUI/HybridSample.MAUI.csproj Updates project references to new library paths under src/Libraries.
samples/HybridSamples/HybridSample.Avalonia/HybridSample.Avalonia.csproj Updates library project reference path and adds an extra package reference.
samples/Directory.Packages.props Adds sample-level central package import that chains to src/Directory.Packages.props.
samples/Directory.Build.props Adds sample-level build props that import src/Directory.Build.props and an optional local override.
samples/Directory.Build.local.props.example Provides a template for developer-local sample build overrides.
samples/Blazing.SubpathHosting.Server/Blazing.SubpathHosting.Server.slnx Adds .slnx entry point for the sample.
samples/Blazing.SubpathHosting.Server/Blazing.SubpathHosting.Server.csproj Updates library project reference path under src/Libraries.
samples/Blazing.Mvvm.Security.Wasm/Blazing.Mvvm.Security.Wasm.slnx Adds .slnx entry point for the sample.
samples/Blazing.Mvvm.Security.Wasm/Blazing.Mvvm.Security.Wasm.csproj Updates library project reference path under src/Libraries.
samples/Blazing.Mvvm.Sample.WebApp/Blazing.Mvvm.Sample.WebApp/Blazing.Mvvm.Sample.WebApp.csproj Updates library project reference path under src/Libraries.
samples/Blazing.Mvvm.Sample.WebApp/Blazing.Mvvm.Sample.WebApp.slnx Adds .slnx entry point (multi-project) for the WebApp sample.
samples/Blazing.Mvvm.Sample.WebApp/Blazing.Mvvm.Sample.WebApp.Client/Blazing.Mvvm.Sample.WebApp.Client.csproj Updates library project reference path under src/Libraries.
samples/Blazing.Mvvm.Sample.Wasm/Blazing.Mvvm.Sample.Wasm.slnx Adds .slnx entry point for the sample.
samples/Blazing.Mvvm.Sample.Wasm/Blazing.Mvvm.Sample.Wasm.csproj Updates library project reference path under src/Libraries.
samples/Blazing.Mvvm.Sample.Server/Blazing.Mvvm.Sample.Server.slnx Adds .slnx entry point for the sample.
samples/Blazing.Mvvm.Sample.Server/Blazing.Mvvm.Sample.Server.csproj Updates library project reference path under src/Libraries.
samples/Blazing.Mvvm.Sample.HybridMaui/Blazing.Mvvm.Sample.HybridMaui.slnx Adds .slnx entry point for the sample.
samples/Blazing.Mvvm.Sample.HybridMaui/Blazing.Mvvm.Sample.HybridMaui.csproj Updates library project reference paths under src/Libraries.
samples/Blazing.Mvvm.AnalyzerTest/README.md Updates analyzer-test documentation to reflect BLAZMVVM0021 scope/behavior.
samples/Blazing.Mvvm.AnalyzerTest/Components/Test021Component.razor Clarifies comment text describing the BLAZMVVM0021 sample pattern.
samples/Blazing.Mvvm.AnalyzerTest/Blazing.Mvvm.AnalyzerTest.slnx Adds .slnx entry point for the analyzer test sample.
samples/Blazing.Mvvm.AnalyzerTest/Blazing.Mvvm.AnalyzerTest.csproj Updates project references to new src/Libraries/* paths.
readme.md Updates TOC formatting, adds/moves links to new samples/ paths, and refreshes documentation sections.
docs/analyzers/BLAZMVVM0021.md Adds new analyzer documentation for BLAZMVVM0021.
ci-cd-test-run.ps1 Updates local CI test runner to use .slnx and new test project paths.
HISTORY.md Reworks TOC formatting and updates sample links to new samples/ paths.
.vscode/settings.json Removes workspace-specific VS Code color customizations.
.github/workflows/release.yml Updates restore/build/test/pack paths to .slnx and new folder layout.
.github/workflows/ci.yml Updates restore/build/test paths to .slnx and new test project locations.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread HISTORY.md Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread docs/analyzers/BLAZMVVM0021.md Outdated
Comment thread docs/analyzers/BLAZMVVM0021.md Outdated
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@mishael-o mishael-o requested a review from gragra33 April 19, 2026 15:15
Copilot AI review requested due to automatic review settings April 19, 2026 15:46

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 29 out of 780 changed files in this pull request and generated 6 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/ci.yml
Comment thread .github/workflows/release.yml
Comment thread README.md
Comment thread README.md
Comment thread README.md
Comment thread docs/analyzers/BLAZMVVM0021.md

@gragra33 gragra33 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@gragra33 gragra33 merged commit e80b6de into develop Apr 27, 2026
9 checks passed
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.

3 participants