-
Notifications
You must be signed in to change notification settings - Fork 717
Introduce the azure publisher #8050
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
Conversation
src/Aspire.Hosting/PublisherDistributedApplicationBuilderExtensions.cs
Outdated
Show resolved
Hide resolved
1c7a9f1 to
91eb64c
Compare
- Add experimental attribute to AddAzurePublisher to that stiches all bicep modules together in main.bicep - Added tests
91eb64c to
eb28a59
Compare
| } | ||
| }; | ||
|
|
||
| // REVIEW: Do we want people to be able to change this |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We could potentially provide an override. I think some people dislike the whole rg- thing (I do, it reminds me of warts on variable names).
But I think we should keep it as default for back compat.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Via options for sure.
Added the azure publisher
main.bicepand to gather all deployment target resources into sub folders that will be deployed separately.