Remove Aspire.Cli from unexpected packages list - #7896
Merged
Merged
Conversation
Mitch Denny (mitchdenny)
requested review from
Eric Erhardt (eerhardt) and
Ankit Jain (radical)
as code owners
March 5, 2025 06:38
Member
Author
|
Ankit Jain (@radical) Jose Perez Rodriguez (@joperezr) the build is currently broken because I set Aspire.Cli to be packable (wasn't picked up on PR validation). Is this the correct approach to solve this problem? |
David Fowler (davidfowl)
approved these changes
Mar 5, 2025
Collaborator
|
Why are these packages still arch specific even though you removed publishaot? |
Member
Author
|
They aren't arch specific (at least Aspire.Cli isn't). It just so happens some of these other ones are. |
github-actions
Bot
added
the
area-integrations
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 subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
Exclude Aspire.Cli from the set of unexpected packages. This is currently breaking the build.