Context
Follow-up from PR #7053 review. Parent tracking issue: #7112
PR Conversations
What needs to change
- Caching: Add caching to the permissions service — permissions are unlikely to change during an azd session.
- Nil checks: Remove unnecessary
nil checks before range statements — range over a nil slice is a no-op in Go.
- Glob matching: Consider using a glob matching library instead of the custom
wildcardMatch implementation to avoid maintaining custom logic if assumptions change.
Files to update
cli/azd/pkg/azapi/permissions.go
Context
Follow-up from PR #7053 review. Parent tracking issue: #7112
PR Conversations
What needs to change
nilchecks beforerangestatements —rangeover a nil slice is a no-op in Go.wildcardMatchimplementation to avoid maintaining custom logic if assumptions change.Files to update
cli/azd/pkg/azapi/permissions.go