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

use dash for 'api_management' #93

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

philippeckelintive
Copy link

Add missing dash for api_management resource.

@Nepomuceno
Copy link
Member

If this is the case there will be a need to also change the regex and do a make all to regenerate the files.

@newice
Copy link

newice commented Aug 4, 2023

Would love to see this PR finished. Even in Microsoft learn the example is apim-navigator-prod.
https://learn.microsoft.com/en-us/azure/cloud-adoption-framework/ready/azure-best-practices/resource-naming#example-names-general

@Nepomuceno
Copy link
Member

In order to finish this pr it would also need to change all the other files not only the json one.
make gen would do it for the person.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants