Skip to content
This repository was archived by the owner on Jul 9, 2025. It is now read-only.

Conversation

@filipelteixeira
Copy link
Contributor

@filipelteixeira filipelteixeira commented Mar 31, 2021

createCosmosDB to createCosmosDb

Description

Found a small typo in the project generated readme file, that prevented the Cosmos DB resource to not be created, if used as described. The value is case sensitive, and is written as createCosmosDB instead of createCosmosDb.

I was provisioning the Azure infrastructure for an article I wrote about MBF and found out the Cosmos DB resource was still being created although I used the correct described value set as false.
After looking into the script code, I realized that the value had a typo, and recreated with the correct parameter name. It worked. :)

Task Item

Didn't found an issue related to it. #minor

Screenshots

@filipelteixeira filipelteixeira changed the title Fixed minor typo in the Composer Project ReadMe file: fix: Fixed minor typo in the Composer Project ReadMe file: Mar 31, 2021
a-b-r-o-w-n
a-b-r-o-w-n previously approved these changes Apr 5, 2021
Copy link
Contributor

@a-b-r-o-w-n a-b-r-o-w-n left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the contribution!

@cwhitten
Copy link
Member

cwhitten commented Apr 7, 2021

@filipelteixeira there is now a conflict on this change. Can you address it?

@ghost
Copy link

ghost commented Apr 7, 2021

CLA assistant check
All CLA requirements met.

@filipelteixeira
Copy link
Contributor Author

@cwhitten reviewed and addressed:

  • Removed the file on the merge overlap;
  • Added the minor fix to the file in the new structure.

@cwhitten
Copy link
Member

cwhitten commented Apr 7, 2021

@filipelteixeira I don't think all the component file renames should be in this diff.

Screen Shot 2021-04-07 at 4 32 31 PM

@filipelteixeira
Copy link
Contributor Author

@filipelteixeira I don't think all the component file renames should be in this diff.

Screen Shot 2021-04-07 at 4 32 31 PM

I'll close this pull request and remake the change onto the new merged main

@filipelteixeira filipelteixeira deleted the filipelteixeira/fix-readme-cosmosdb-parameter branch April 7, 2021 23:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants