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.

Sign up for GitHub

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

Jump to bottom

Add tests using a const instance of the AppConfig client #6333

Open
ahsonkhan opened this issue Jan 11, 2025 · 1 comment
Open

Add tests using a const instance of the AppConfig client #6333

ahsonkhan opened this issue Jan 11, 2025 · 1 comment
Assignees
@gearama
Labels
App Configuration Azure.ApplicationModel.Configuration test-enhancement
Milestone
2025-04

Comments

@ahsonkhan
Copy link
Contributor

This is to ensure methods for each service operation are marked as const and callable from a const reference.

From #6328 (comment)
@ahsonkhan ahsonkhan added App Configuration Azure.ApplicationModel.Configuration test-enhancement labels Jan 11, 2025
@ahsonkhan ahsonkhan added this to the 2025-03 milestone Jan 11, 2025
@ahsonkhan ahsonkhan self-assigned this Jan 11, 2025
@ahsonkhan ahsonkhan mentioned this issue Jan 11, 2025
Merged
@RickWinter RickWinter added Azure.Identity and removed App Configuration Azure.ApplicationModel.Configuration labels Jan 24, 2025
@antkmsft
Copy link
Member

@RickWinter, I think this is AppConfig, not Identity

@antkmsft antkmsft added App Configuration Azure.ApplicationModel.Configuration and removed Azure.Identity labels Feb 12, 2025
@antkmsft antkmsft assigned gearama and unassigned antkmsft Feb 12, 2025
@RickWinter RickWinter modified the milestones: 2025-03, 2025-04 Feb 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gearama gearama

Labels
App Configuration Azure.ApplicationModel.Configuration test-enhancement
Projects
None yet
Milestone
2025-04
Development

No branches or pull requests
4 participants
@RickWinter @ahsonkhan @antkmsft @gearama