Skip to content
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

Implement IResourceWithArgs on ProjectResource #3545

Merged
merged 1 commit into from
Apr 10, 2024

Conversation

eerhardt
Copy link
Member

@eerhardt eerhardt commented Apr 10, 2024

Also fix the command line arg parsing to be the same as .NET Process's:

  • double quotes around single arguments
  • consecutive double quotes inside a quoted argument means one double quote

Fix #3306

"args" on a project resource doesn't work with azd yet. It doesn't error, but the project's container doesn't get started with the args. Logged Azure/azure-dev#3678

cc @vhvb1989

Microsoft Reviewers: Open in CodeFlow

Also fix the command line arg parsing to be the same as .NET Process's:
* double quotes around single arguments
* consecutive double quotes inside a quoted argument means one double quote

Fix dotnet#3306
@dotnet-issue-labeler dotnet-issue-labeler bot added the area-app-model Issues pertaining to the APIs in Aspire.Hosting, e.g. DistributedApplication label Apr 10, 2024
@eerhardt eerhardt merged commit e40f73a into dotnet:main Apr 10, 2024
8 checks passed
@eerhardt eerhardt deleted the CommandLineArgs branch April 10, 2024 14:43
@eerhardt
Copy link
Member Author

/backport to release/8.0

Copy link
Contributor

Started backporting to release/8.0: https://github.com/dotnet/aspire/actions/runs/8633209885

@github-actions github-actions bot locked and limited conversation to collaborators May 11, 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.

Implement IResourceWithArgs on ProjectResource
2 participants