Repository navigation
Post-merge cleanup: dependency bot model pin, Mac Catalyst 17.0 for the MAUI libraries - #50
Merged
Merged
Conversation
…s rules The compiled default model, claude-sonnet-4.6, is no longer offered to the Copilot CLI, so every weekly run since 2026-08-31 failed with a 400. The Avalonia 11.3 rule is replaced: the repo is on Avalonia 12 without Avalonia.ReactiveUI. New rules hold ReactiveUI on 24.x and keep the packages it requires in step, which is what broke restore in #49. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
The Xcode 27 SDK rejects app heads below Mac Catalyst 17.0, so no app built with the current workload can target the 15.0 the libraries declared. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Contributor
There was a problem hiding this comment.
Copilot review overview
🟡 Changes recommended
The SourceGenerators rule lacks a valid version boundary, and the review record uses a future timestamp.
Review effort: Balanced
Findings: 1
What changed in this PR
Updates dependency automation and aligns MAUI libraries with Xcode 27 requirements.
Changes:
- Pins the dependency bot to
claude-sonnet-5and updates its rules. - Raises Mac Catalyst minimum support to 17.0.
- Adds the October 2 PR review record.
| File | Description |
|---|---|
.github/workflows/dependency-update.md |
Updates model and dependency rules. |
.github/workflows/dependency-update.lock.yml |
Regenerates the compiled workflow. |
Stellar.Maui/Stellar.Maui.csproj |
Raises Mac Catalyst minimum to 17.0. |
Stellar.Maui.PopUp/Stellar.Maui.PopUp.csproj |
Raises Mac Catalyst minimum to 17.0. |
pr-review-2026-10-02.md |
Records recent PR review findings. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Comment on lines
+69
to
+72
| **Keep ReactiveUI on the 24.x line.** This covers every `ReactiveUI.*.Reactive` | ||
| package and `ReactiveUI.SourceGenerators`. ReactiveUI 25 moves binding onto a | ||
| source generator: `Bind` calls through private view fields, which is how Stellar | ||
| views are written, get no generated binding and throw at run time, and the |
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.
What
claude-sonnet-5. The compiler's default,claude-sonnet-4.6, is no longer offered, so every weekly run since 2026-08-31 failed with a 400 (run 36453567043; its log lists the available models).dependency-update.lock.ymlis recompiled with gh-aw v0.77.5, the version it was already on.Avalonia.ReactiveUI), adds "keep ReactiveUI on 24.x", and adds a rule to keepMicrosoft.Extensions.*/ MAUI pins in step with what a ReactiveUI update requires (the NU1109 failure hit in Fix develop build and CI, bump ReactiveUI to 24.3.0 #49).Stellar.MauiandStellar.Maui.PopUpminimum 15.0 → 17.0, matching what the Xcode 27 SDK enforces for app heads. This drops Mac Catalyst 15 and 16 for consumers of those packages.pr-review-2026-10-02.md, the record of today's review of Fix build break from auto-fix commits (v2.1.3 release failed; 2.1.3 partially published) #38–chore(deps): update NuGet packages (patch/minor) #41, Improves ItemsView recycling performance on Android #47 and fix(maui): keep BindCommand bound when a view's view model is replaced #48.Verification
gh aw compile: 0 errors, 0 warnings.🤖 Generated with Claude Code