microsoft / microsoft/aspire

Group AddAzureFunctionTool parameters into an options type

Open
#16,610 1 comment 0 reactions 0 assignees View on GitHub
area-integrations
Dominant language
C#
Stars
6.3k
Forks
991
Avg merge
2d 15h
Merged PRs (30d)
196

Description

## Background

This tracks a review comment from #16042: https://github.com/microsoft/aspire/pull/16042#discussion_r3163355167

`AddAzureFunctionTool(...)` currently exposes eight user-supplied parameters, including paired queue endpoint/name values. That makes call sites hard to read and the API hard to evolve without adding more overloads.

## Proposal

Consider grouping the Azure Function tool configuration into an options/configuration type before this preview API ships stable.

Reference: https://learn.microsoft.com/dotnet/standard/design-guidelines/parameter-design

## Source

File: `src/Aspire.Hosting.Foundry/api/Aspire.Hosting.Foundry.cs`
PR: #16042
Comment: https://github.com/microsoft/aspire/pull/16042#discussion_r3163355167

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.