Skip to content

Update launchSettings.json to support manifest generation #9959

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jun 19, 2025

Conversation

sebastienros
Copy link
Member

This launchSettings.json file has a wrong verb and the refreshManifests.ps1 script complains about it missing

This launchSettings.json file has a wrong verb and the `refreshManifests.ps1` script complains about it missing
@sebastienros sebastienros requested review from Copilot and radical June 19, 2025 18:29
@github-actions github-actions bot added the area-app-model Issues pertaining to the APIs in Aspire.Hosting, e.g. DistributedApplication label Jun 19, 2025
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the launchSettings.json to use the correct key ("generate-manifest") that is compatible with the manifest generation script.

  • Renamed "publish-manifest" to "generate-manifest" in launchSettings.json
  • Fixes the issue where refreshManifests.ps1 complains about a missing manifest key

@sebastienros sebastienros assigned davidfowl and unassigned davidfowl Jun 19, 2025
@davidfowl davidfowl merged commit 2c89481 into main Jun 19, 2025
253 checks passed
@davidfowl davidfowl deleted the sebros/manifest branch June 19, 2025 19:21
captainsafia pushed a commit that referenced this pull request Jun 20, 2025
This launchSettings.json file has a wrong verb and the `refreshManifests.ps1` script complains about it missing
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
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