Skip to content

Fixed not resolving bicep outputs when a lambda was used#2805

Merged
davidfowl merged 3 commits intomainfrom
davidfowl/fix-bicep-manifest
Mar 12, 2024
Merged

Fixed not resolving bicep outputs when a lambda was used#2805
davidfowl merged 3 commits intomainfrom
davidfowl/fix-bicep-manifest

Conversation

@davidfowl
Copy link
Copy Markdown
Contributor

@davidfowl davidfowl commented Mar 12, 2024

  • Added tests
Microsoft Reviewers: Open in CodeFlow

@ghost ghost added the area-app-model Issues pertaining to the APIs in Aspire.Hosting, e.g. DistributedApplication label Mar 12, 2024
var executionContext = new DistributedApplicationExecutionContext(DistributedApplicationOperation.Publish);
writer.WriteStartObject();
await ManifestPublisher.WriteResourceAsync(resource, new ManifestPublishingContext(executionContext, Environment.CurrentDirectory, writer));
await ManifestPublisher.WriteResourceAsync(resource, new ManifestPublishingContext(executionContext, Path.Combine(Environment.CurrentDirectory, "manifest.json"), writer));
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@eerhardt this caused the net8 issue 😄

@davidfowl davidfowl enabled auto-merge (squash) March 12, 2024 13:56
@davidfowl davidfowl merged commit 6fa35f1 into main Mar 12, 2024
@davidfowl davidfowl deleted the davidfowl/fix-bicep-manifest branch March 12, 2024 14:56
@github-actions github-actions bot locked and limited conversation to collaborators Apr 18, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-app-model Issues pertaining to the APIs in Aspire.Hosting, e.g. DistributedApplication

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants