Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
17 changes: 15 additions & 2 deletions .github/skills/awesome-azd-template-submit/SKILL.md
Original file line number Diff line number Diff line change
Expand Up @@ -49,6 +49,10 @@ Only the **Source Repository** field is required. All other fields are
auto-detected by the pipeline. Do NOT fill in optional fields unless the user
explicitly provides overrides.

For author attribution, use the source repository owner as the default author.
Use their GitHub profile as the author URL unless the submitter explicitly
declares a different author URL.

```bash
gh issue create \
--repo Azure/awesome-azd \
Expand Down Expand Up @@ -118,13 +122,22 @@ After filing, report the issue URL and explain that:

Use the **Trusted Author** field only when the GitHub issue creator is a trusted
author. Trusted authors do not need manual template testing requested from
@v-xuto or deployment proof requested from the PR author.
@HarveyLink or deployment proof requested from the PR author.

Current trusted authors:
- paulyuk

Before requesting testing, check whether the template submission issue was
created by a trusted author. Do not request manual testing or deployment proof
for trusted authors.

For community templates (those without the `msft` tag), comment on the PR and

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This changes repository-wide submission policy in a PR whose stated purpose is adding one gallery template. The skill already existed, but these new attribution and testing rules affect every future submission and need their own review. The testing step also cannot run reliably in this workflow because the PR is created asynchronously after the skill files the issue.

Could we remove the SKILL.md changes from this PR and propose them separately, with a clear follow-up mechanism and the correct person to contact for Microsoft templates?

mention @HarveyLink to request manual testing. For Microsoft templates (those with
the `msft` tag), ask the PR author for a test pipeline link or other proof of
successful deployment.

When mentioning people in issue or PR comments, write the mention as plain text
such as @v-xuto. Do not wrap mentions in inline code formatting, because that
such as @HarveyLink. Do not wrap mentions in inline code formatting, because that
prevents GitHub notifications.

## Handling user overrides
Expand Down
30 changes: 30 additions & 0 deletions website/static/templates.json
Original file line number Diff line number Diff line change
Expand Up @@ -8210,5 +8210,35 @@
"terraform"
],
"id": "64577715-e9a4-4a65-a514-4233763474c2"
},
{
"title": "Text Semantic Search using embedding models in Microsoft Foundry and Azure PostgreSQL",
"description": "A .NET application that generates text embeddings with Microsoft Foundry, stores them in Azure Database for PostgreSQL with pgvector, and searches for semantically similar text. It provisions a Blazor Server frontend and Azure Functions API using Bicep.",
"preview": "./templates/images/db-semantic-search-architecture.png",
"authorUrl": "https://info.massimobonanni.it/",
"author": "Massimo Bonanni",
"source": "https://github.com/massimobonanni/DBSemanticSearch",
"tags": [
"community",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot New community template contribution should ask for testing. Update that into skill and ping v-xuto.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Updated the submission skill to request manual testing for community templates, except for submissions from trusted authors. Tong Xu (MSFT) (@v-xuto) Please test this community template to ensure it works as expected. The guidance and requested author URL are committed as e768e9f.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Updated the testing contact in the skill and committed as COMMIT_HASH. Minghao Chen (@HarveyLink) Please test this community template before merge.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This is a community template, and the submission issue author is not on the trusted-author list. Could we add the required manual testing request before merge?

Tong Xu (MSFT) (@v-xuto) Please test this community template to ensure it works as expected.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hi Kristen Womack (@kristenwomack), we have tested this community template, and its core functionality works as expected.

"new",
"ai"
],
"languages": [
"dotnetCsharp"
],
"frameworks": [
"blazor"
],
"azureServices": [
"appservice",
"functions",
"aifoundry",
"azuredb-postgreSQL",
"appinsights"
],
"IaC": [
"bicep"
],
"id": "6522cdb6-4c9d-4d77-84dd-746daf408966"
}
]
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading