Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Split out build step in publisher.#8567

Merged
mitchdenny merged 1 commit into
mainfrom
mitchdenny/perf-work-1
Apr 5, 2025
Merged

Split out build step in publisher.#8567
mitchdenny merged 1 commit into
mainfrom
mitchdenny/perf-work-1

Conversation

@mitchdenny

@mitchdenny mitchdenny commented Apr 5, 2025

Copy link
Copy Markdown
Member

This PR splits out the build in the publisher to make it more obvious that publish involves doing a build. This is the make sure people don't think getting the list of publishers takes qutie so long and know its a function of build time.

aspire-build-with-build-status.mp4

Copilot AI review requested due to automatic review settings April 5, 2025 08:59

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

Comment thread src/Aspire.Cli/DotNetCliRunner.cs
@mitchdenny mitchdenny self-assigned this Apr 5, 2025
@mitchdenny mitchdenny added this to the 9.2 milestone Apr 5, 2025
@mitchdenny mitchdenny requested a review from davidfowl April 5, 2025 09:00
@mitchdenny mitchdenny merged commit 5becd83 into main Apr 5, 2025
@mitchdenny mitchdenny deleted the mitchdenny/perf-work-1 branch April 5, 2025 09:16
@mitchdenny

Copy link
Copy Markdown
Member Author

/backport to release/9.2

@github-actions

github-actions Bot commented Apr 5, 2025

Copy link
Copy Markdown
Contributor

Started backporting to release/9.2: https://github.com/dotnet/aspire/actions/runs/14280509123

@github-actions github-actions Bot locked and limited conversation to collaborators May 6, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants