terraform-provider-azurecaf icon indicating copy to clipboard operation
terraform-provider-azurecaf copied to clipboard

Correct slugs according to caf recommended abbreviations

Open t3mi opened this issue 2 years ago • 4 comments

Fixes #186, #168

PR Checklist


  • [x] I have read the CONTRIBUTING.MD instructions
  • [x] I have changed the resourceDefinition.json
  • [x] I have generated the resource model (there's a models_generated.go file in my PR)
  • [x] I have updated the README.md#resource-status
  • [x] I have checked to ensure there aren't other open Pull Requests for the same update/change?

Description

Does this introduce a breaking change

  • [x] YES
  • [ ] NO

Testing

t3mi avatar Sep 06 '22 10:09 t3mi

Very good catches @t3mi - we will add it on next major update to avoid regression on already deployed resources!

arnaudlh avatar Sep 28 '22 04:09 arnaudlh

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

sonarqubecloud[bot] avatar Sep 28 '22 04:09 sonarqubecloud[bot]

Yeah, I've targeted 3_0 branch for the merge. If this isn't right please provide the correct target. Thanks!

t3mi avatar Sep 28 '22 11:09 t3mi

@arnaudlh , could this be part of the 2.0 release? Maybe generating a -preview4; I am considering using this provider, but I'd like to start CAF compliant

carnei-ro avatar Jul 04 '24 12:07 carnei-ro