cheapnud/CheapHelpers
Watch
1
0
Fork
You've already forked CheapHelpers
 0
Code Issues 8 Pull requests Projects Releases Packages Wiki Activity Actions

Ship the Blazor static assets with the props manifest again #90

Merged
cheapnud merged 2 commits from b/89-static-asset-manifest into master 2026-10-22 19:02:39 +00:00
Conversation 1 Commits 2 Files changed 2 +7 -1
cheapnud commented 2026-10-22 19:01:11 +00:00
Owner
Copy link

Packs CheapHelpers.Blazor 4.0.1 with the props-based static web asset manifest so published apps with a same-named wwwroot file no longer crash at startup.

Closes #89

Packs CheapHelpers.Blazor 4.0.1 with the props-based static web asset manifest so published apps with a same-named wwwroot file no longer crash at startup. Closes #89
Note the 4.0.1 static asset fix in the README
All checks were successful
Claude PR Review / AI Code Review (pull_request) Successful in 12s
Details
Build / Build and Test (pull_request) Successful in 25s
Details
2fc68c1266
forgejo-actions commented 2026-10-22 19:01:27 +00:00
Copy link

🤖 Claude AI Code Review

Summary
Bumps CheapHelpers.Blazor to 4.0.1 and switches packaging back to the props-based static web asset manifest. This fixes a startup crash in published apps that have a wwwroot file with the same name as one of the package's assets (#89). The README is updated to match.

Code Quality
⭐⭐⭐⭐⭐
A small, targeted fix with an inline comment that explains why the SDK property is overridden, which will help anyone who touches it later.

Verdict
APPROVE

📊 Tokens: 3169 input + 492 output | 💰 Cost: ~$0.0225
Generated by Claude Opus 5.5 via Forgejo Actions

## 🤖 Claude AI Code Review **Summary** Bumps CheapHelpers.Blazor to 4.0.1 and switches packaging back to the props-based static web asset manifest. This fixes a startup crash in published apps that have a `wwwroot` file with the same name as one of the package's assets (#89). The README is updated to match. **Code Quality** ⭐⭐⭐⭐⭐ A small, targeted fix with an inline comment that explains why the SDK property is overridden, which will help anyone who touches it later. **Verdict** **APPROVE** --- 📊 **Tokens**: 3169 input + 492 output | 💰 **Cost**: ~$0.0225 *Generated by Claude Opus 5.5 via Forgejo Actions*
cheapnud merged commit 8ae56f6966 into master 2026-10-22 19:02:39 +00:00
cheapnud deleted branch b/89-static-asset-manifest 2026-10-22 19:02:39 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
backend
Server-side / API / services

  
blocked
Blocked on a dependency

  
bug
Something is broken

  
database
Schema, migrations, queries

  
deferred
Consciously postponed

  
desktop
WPF / Avalonia / MAUI desktop

  
docs
Documentation

  
draft
Not clear yet, needs refining

  
duplicate
Already tracked elsewhere

  
epic
Larger work spanning multiple tasks/ops

  
firmware
Device / embedded / Rust gateway

  
frontend
UI / Blazor / views

  
high
High priority

  
hotfix
Urgent, fix now

  
idea
Someday / maybe, not committed

  
in-progress
Actively being worked

  
invalid
Not a real issue

  
low
Low priority / someday

  
mobile
Android / mobile

  
ops
Infra, deployment, tooling, maintenance

  
task
Coding work

  
wontfix
Will not be worked on

No labels
backend
blocked
bug
database
deferred
desktop
docs
draft
duplicate
epic
firmware
frontend
high
hotfix
idea
in-progress
invalid
low
mobile
ops
task
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
cheapnud/CheapHelpers!90
No description provided.