finos / terraform-aws-cfi-eks

Apache License 2.0
0 stars 1 forks source link

Rename repo to `cfi-terraform-aws-eks` #4

Closed mcleo-d closed 2 years ago

mcleo-d commented 2 years ago

Description

Can this repo be renamed to cfi-terraform-aws-eks to ensure it's searchable and filterable under the cfi namespace for Compliant Financial Infrastructure?

Kind regards,

James.

eddie-knight commented 2 years ago

I don't believe we'll be able to achieve the desired functionality with the proposed name change.

Suggest closing this issue and tagging it as invalid on the next community call.

mcleo-d commented 2 years ago

Hey @eddie-knight - Lets discuss here https://github.com/finos/compliant-financial-infrastructure/issues/265

mcleo-d commented 2 years ago

@eddie-knight - I've added the invalid label and added to ready for review for project call discussion and closure.

mcleo-d commented 2 years ago

Issue closed on https://github.com/finos/compliant-financial-infrastructure/issues/265

For more information see https://www.terraform.io/registry/modules/publish#requirements

mcleo-d commented 2 years ago

Reopened on https://github.com/finos/compliant-financial-infrastructure/issues/265 😸

thinkl33t commented 2 years ago

To put this into the terraform repos, we need to stick with the correct nomenclature terraform-nameofcloudprovider-freetext, so we can't prepend with cfi- as initially suggested.

The current name of this module however will clash with more generic aws eks modules in the repo. @ml4 suggested we used terraform-aws-cfi-eks to distinguish from these, and build a naming convention around this style.

eddie-knight commented 2 years ago

Agreed, happy to apply the name change to terraform-aws-cfi-eks.

@TheJuanAndOnly99, could you action the rename please?

TheJuanAndOnly99 commented 2 years ago

@eddie-knight done! please check and close :)

eddie-knight commented 2 years ago

Thanks @TheJuanAndOnly99 !!