Skip to content

Conversation

@im-samz
Copy link

@im-samz im-samz commented Jun 16, 2025

Please fill out this template! There are three different types of contributions, feel free to delete the checklists that are not applicable to your contribution type.

If you are submitting a new azd template to the gallery

Fill this out if you want your template to be added to the awesome-azd gallery!

Your template repository

Place your template repository link here: https://github.com/Azure-Samples/functions-quickstart-dotnet-azd-timer

  • Added an entry to https://github.com/Azure/awesome-azd/blob/main/website/static/templates.json 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 - The image should include all services and their connections (example). You should add the image to the website/static/templates/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 - Specify tags to represent the template. If you don't see a relevant tag for your template? Feel free to add one!

    • Languages Tags - Tags for languages representing the template.

    • Azure Services Tags - Tags for azure services representing the template.

    • ID - Please provide a unique uuid.

      Required tags:

      • Tag your template as Microsoft-authored ("msft") or Community-authored ("community")
      • Tag the IaC provider ("bicep" or "terraform")
      • Add the "new" tag for any newly authored templates
  • 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.

  • If the template is Microsoft-authored, we encourage you to also publish it to learn.microsoft.com/samples.

@im-samz
Copy link
Author

im-samz commented Jun 16, 2025

PR that made my repo azd compatible: Azure-Samples/functions-quickstart-dotnet-azd-timer#1

@hemarina
Copy link
Contributor

@v-xuto Could you test this template?

@v-xuto
Copy link
Member

v-xuto commented Jun 17, 2025

@v-xuto Could you test this template?

Testing

@im-samz
Copy link
Author

im-samz commented Jun 25, 2025

Hey @v-xuto, I've addressed the lanuages comment from @hemarina, and my recent PR to the Azure-samples repo should address your issues. Let me know if there's anything else I should do.

@v-xuto
Copy link
Member

v-xuto commented Jun 27, 2025

Hey @v-xuto, I've addressed the lanuages comment from @hemarina, and my recent PR to the Azure-samples repo should address your issues. Let me know if there's anything else I should do.

@im-samz Issue3 has been resolved, but there are still some minor issues with Issue4 and Issue5. Comments have been added to the relevant issues, please review them.

Copy link
Contributor

@hemarina hemarina left a comment

Choose a reason for hiding this comment

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

Thank you for your contribution!

@spboyer spboyer requested a review from Copilot June 30, 2025 20:30
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a new Azure Functions C# Timer Trigger quickstart template to the Azure Developer CLI gallery.

  • Introduces a new template entry in templates.json for a functions timer trigger sample
  • Specifies metadata including title, description, preview image, author info, tags, and services
  • Assigns a unique UUID for the new template
Comments suppressed due to low confidence (2)

website/static/templates.json:3270

  • The preview path uses a placeholder test.png. Please update this to a descriptive filename and ensure the corresponding image is committed under website/static/templates/images.
    "preview": "./templates/images/test.png",

website/static/templates.json:3274

  • Confirm that the "dotnetCsharp" language tag matches the existing language tags in tags.tsx (e.g., it may need to be "csharp").
    "languages": [

changed "insure" to "ensure"

Co-authored-by: Copilot <[email protected]>
@hemarina hemarina requested a review from Copilot October 29, 2025 21:49
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comment on lines +3267 to +3289
{
"title": "Azure Functions C# Timer Trigger using Azure Developer CLI",
"description": "This repository contains an Azure Functions Timer trigger quickstart written in C# and deployed to Azure Functions Flex Consumption using the Azure Developer CLI (AZD). This sample uses managed identity and a virtual network to ensure it's secure by default.",
"preview": "./templates/images/test.png",
"authorUrl": "https://github.com/Azure-Samples",
"author": "Azure Functions Team",
"source": "https://github.com/Azure-Samples/functions-quickstart-dotnet-azd-timer",
"languages": [
"dotnetCsharp"
],
"tags": [
"msft",
"bicep",
"new"
],
"azureServices": [
"functions",
"managedidentity",
"vnets",
"appinsights"
],
"id": "b7e6c2e2-5e2e-4c8e-8e6e-2e2c8e5e6c2e"
},
Copy link

Copilot AI Oct 29, 2025

Choose a reason for hiding this comment

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

The field ordering is inconsistent with other entries in this file. Compare with surrounding entries (lines 3203-3224, 3225-3246, 3247-3266) which follow the pattern: title, description, preview, authorUrl, author, source, tags, azureServices, id, languages. This entry has 'languages' before 'tags'. For consistency and maintainability, reorder the fields to match the established pattern.

Copilot uses AI. Check for mistakes.
Copy link
Contributor

Choose a reason for hiding this comment

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

Could you move these changes @copilot to the bottom of the file (i.e the last item in array)?

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.

4 participants