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.

Sign up for GitHub

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

Jump to bottom

Add cookiecutter templates #196

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Add cookiecutter templates #196

wants to merge 3 commits into from

Conversation

pamelafox
Copy link
Member

17 repos are being added in this PR. They were all generated by https://github.com/kjaymiller/cookiecutter-relecloud so they have a lot of code (especially Bicep/devcontainer) in common.

  • Added an entry to https://github.com/Azure/awesome-azd/blob/main/website/src/data/users.tsx that includes:

    • Template title - A short title that reflects the local application stack that someone could use to get their application on Azure (e.g. "Containerized React Web App with Java API and MongoDB")
    • Description - 1-2 sentence description of the architecture (e.g. Azure services) or solution that is defined by the template.
    • Architecture Diagram or Application Screenshot - Used as display image for gallery card. The image should include all services and their connections (example). You should add the image to the website/src/data/images/.
    • Link to Author's GitHub or other relevant website - Used for attribution
    • Author's Name - Name to credit on the gallery card
    • Link to template source - Link to the template GitHub repo
    • Tags - One or more tags representing the template. Provide at least 1 tag for programming language used and at least 1 tag for Azure services integrated. Also tag the IaC provider (Bicep or Terraform). If you don't see a relevant tag for your template? Feel free to add one!

  • In the PR comment, if you can also add a link to the PR where you made your repo azd compatible this will allow us to provide feedback on your template and speed up the review process.

@pamelafox pamelafox mentioned this pull request Sep 27, 2023
Open
17 tasks
gkulin
gkulin approved these changes Sep 28, 2023
View reviewed changes
Copy link
Collaborator

@gkulin gkulin left a comment

Choose a reason for hiding this comment

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

thanks Pamela I'm excited to get all of these included! 🙂

@pamelafox
Copy link
Member Author

Thanks for approval! FYI that I don't have merge permission, so someone on your team will need to merge when you're ready.

@hemarina
Copy link
Contributor

hemarina commented Oct 2, 2023 •
edited
Loading

Thanks Pamela. I'm playing around with these templates and is checking if these templates need to be tested.

@gkulin
Copy link
Collaborator

gkulin commented Oct 3, 2023

@v-xuto could you test these templates? I spoke with Pamela and they have done some testing and they should all successfully deploy but mostly want to check for cross-os issues

@v-xuto
Copy link
Member

v-xuto commented Oct 7, 2023

@v-xuto could you test these templates? I spoke with Pamela and they have done some testing and they should all successfully deploy but mostly want to check for cross-os issues

@gkulin Sure. We will start test these templates.

@v-xuto
Copy link
Member

v-xuto commented Oct 16, 2023

@gkulin, @pamelafox We have finished these template test, and we filed eight issues. Please review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gkulin gkulin gkulin approved these changes

@hemarina hemarina hemarina approved these changes

@rajeshkamal5050 rajeshkamal5050 Awaiting requested review from rajeshkamal5050 rajeshkamal5050 is a code owner

Assignees

@pamelafox pamelafox

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@pamelafox @hemarina @gkulin @v-xuto