Skip to content

Update Aspire AppHost SDK to 13.5.4 - #192

Closed
dukesteen wants to merge 1 commit into
mainfrom
aspire-sdk-13.5.4
Closed

Update Aspire AppHost SDK to 13.5.4#192
dukesteen wants to merge 1 commit into
mainfrom
aspire-sdk-13.5.4

Conversation

@dukesteen

@dukesteen dukesteen commented Sep 30, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Summary

  • Bumps Aspire.AppHost.Sdk from 13.4.6 to 13.5.4 in both sample AppHosts (samples/Aspire and samples/DistributedAspire), matching the Aspire.Hosting.* 13.5.4 packages already pinned in Directory.Packages.props.
  • Suppresses ASPIRE010, which the 13.5 SDK emits because it defaults AspireUseCliBundle to false. The samples keep using the packaged DCP and dashboard, so they don't need the Aspire CLI installed. Enabling the bundle can happen in a separate change.

Testing

  • Both AppHosts build with 0 warnings and 0 errors.
  • Started samples/Aspire/AppHost (net10.0) locally: the dashboard came up on :15888 and it logged "Distributed application started".

@dukesteen
Update Aspire AppHost SDK to 13.5.4
1ef6f27 
Aligns Aspire.AppHost.Sdk with the 13.5.4 Aspire.Hosting packages and
suppresses the new ASPIRE010 warning, since the samples keep using the
packaged DCP and dashboard rather than the Aspire CLI bundle.
@coderabbitai

coderabbitai Bot commented Sep 30, 2026

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 49 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 9aeef6f7-446e-4763-a7d5-695679585a72

📥 Commits

Reviewing files that changed from the base of the PR and between 517a73d and 1ef6f27.

📒 Files selected for processing (2)
  • samples/Aspire/AppHost/Immediate.Jobs.Aspire.AppHost.csproj
  • samples/DistributedAspire/AppHost/Immediate.Jobs.DistributedAspire.AppHost.csproj
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Autopilot is currently an internal CodeRabbit preview.

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.

@dukesteen

dukesteen commented Sep 30, 2026

Copy link
Copy Markdown
Contributor Author

Closing: the local error was a stale database volume, not the SDK version. Versions will be aligned in a separate PR.

@dukesteen dukesteen closed this Sep 30, 2026
@dukesteen
dukesteen deleted the aspire-sdk-13.5.4 branch September 30, 2026 13:59
@coveralls

coveralls commented Sep 30, 2026

Copy link
Copy Markdown

Coverage Report for CI Build 36724800185

Coverage increased (+0.3%) to 89.045%

Details

  • Coverage increased (+0.3%) from the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.

Coverage Stats

Coverage Status
Relevant Lines: 13410
Covered Lines: 11941
Line Coverage: 89.05%
Coverage Strength: 2.88 hits per line
💛 - Coveralls

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@dukesteen @coveralls