Skip to content

Add missing installation guide to Brouter demo (#12731)#12732

Merged
msynk merged 1 commit into
bitfoundation:developfrom
msynk:12731-brouter-demo-installation-guides
Jul 21, 2026
Merged

Add missing installation guide to Brouter demo (#12731)#12732
msynk merged 1 commit into
bitfoundation:developfrom
msynk:12731-brouter-demo-installation-guides

Conversation

@msynk

@msynk msynk commented Jul 20, 2026

Copy link
Copy Markdown
Member

closes #12731

Summary by CodeRabbit

  • Documentation
    • Updated getting-started instructions with explicit NuGet commands for installing Bit.Brouter.
    • Added optional setup instructions for Bit.Brouter.Generators.
    • Included package reference examples with version 10.5.0 and generator asset configuration.
    • Refined project setup guidance for adding routing and code generation to Blazor applications.

@msynk
msynk requested a review from yasmoradi July 20, 2026 18:58
@coderabbitai

coderabbitai Bot commented Jul 20, 2026

Copy link
Copy Markdown

Review Change Stack

Walkthrough

The Brouter demo pages now include explicit NuGet commands and PackageReference examples for Bit.Brouter and the optional Bit.Brouter.Generators package, using version 10.5.0 and generator package isolation.

Changes

Brouter installation documentation

Layer / File(s) Summary
Package installation examples
src/Brouter/Demos/Core/Pages/HomePage.razor, src/Brouter/Demos/Core/Pages/GeneratorPage.razor
Getting-started instructions now show package installation commands and PackageReference examples for Bit.Brouter and Bit.Brouter.Generators; the generator reference uses PrivateAssets="all".
Estimated code review effort: 1 (Trivial) ~5 minutes

Suggested reviewers: yasmoradi

Poem

A rabbit hops through NuGet snow,
With packages ready, set to go.
Brouter guides now clearly gleam,
Generators join the setup stream.
10.5.0—a carrot dream!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly states the main change: adding installation guidance to the Brouter demo.
Linked Issues check ✅ Passed The changes add installation instructions to both demo pages, matching issue #12731.
Out of Scope Changes check ✅ Passed The edits stay within documentation changes for the demo pages and add no unrelated runtime behavior.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

@coderabbitai coderabbitai Bot 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.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@src/Brouter/Demos/Core/Pages/HomePage.razor`:
- Around line 22-31: Align both installation examples with the pinned package
version: in src/Brouter/Demos/Core/Pages/HomePage.razor lines 22-31 and
src/Brouter/Demos/Core/Pages/GeneratorPage.razor lines 30-38, add the 10.5.0
version option to both Bit.Brouter and Bit.Brouter.Generators CLI commands,
leaving the existing PackageReference versions unchanged.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: ff248332-7522-496f-b97b-c77d49039df9

📥 Commits

Reviewing files that changed from the base of the PR and between c45ae26 and 11810c3.

📒 Files selected for processing (2)
  • src/Brouter/Demos/Core/Pages/GeneratorPage.razor
  • src/Brouter/Demos/Core/Pages/HomePage.razor

Comment thread src/Brouter/Demos/Core/Pages/HomePage.razor
@msynk
msynk merged commit 5200815 into bitfoundation:develop Jul 21, 2026
3 checks passed
@msynk
msynk deleted the 12731-brouter-demo-installation-guides branch July 21, 2026 06:10
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.

Missing installation instructions from the demo pages of Brouter

1 participant