Azure-Samples / azure-search-openai-demo-csharp

A sample app for the Retrieval-Augmented Generation pattern running in Azure, using Azure Cognitive Search for retrieval and Azure OpenAI large language models to power ChatGPT-style and Q&A experiences.
MIT License
549 stars 344 forks source link

Acceptance Criteria Checklist (DoD) #320

Open anfibiacreativa opened 1 month ago

anfibiacreativa commented 1 month ago

The following checklist must be complete before a template is published.

Repository Management

Source code structure and conventions

Functional requirements

In the absence of e2e tests,

Security requirements

When a service selected doesn't support Managed Identity, the corresponding issue must have been reported and the security considerations section in the readme, should clearly explain the alternatives.

The following items are not strictly enforced but may prevent the template from being added to the gallery.

Project code follows standard structure, per language. Please check one.

Code follows recommended style guide

anfibiacreativa commented 1 month ago

@luisquintanilla Please review the pending items or assign to corresponding contributor to help close the issue. Thank you!

luisquintanilla commented 1 month ago

@LittleLittleCloud can you please take a look when you get a chance. Thanks!

LittleLittleCloud commented 1 month ago

@anfibiacreativa The secruity guideline link is broken, can you fix that

anfibiacreativa commented 1 month ago

@anfibiacreativa The secruity guideline link is broken, can you fix that

Hi. That's because it is not in place. It should be added to the root.

LittleLittleCloud commented 1 month ago

@anfibiacreativa I'm refering to this link. which gives me a 404...

Would you share an example of SECURITY.MD that I can added to the repo

image