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

This repo is the Java version of Microsoft's sample app for ChatGPT + Enterprise data.
MIT License
74 stars 76 forks source link

[Auto] AI Gallery Standard Validation #90

Closed ai-apps-bot closed 1 month ago

ai-apps-bot commented 2 months ago

AI Gallery Standard Validation: PASSED

Repository Management:

Source code structure and conventions:

Functional Requirements:

How to fix?

The full Definition of Done of the AI-Gallery template and fix approached can be found HERE.

dantelmomsft commented 1 month ago

I've pushed a ⁠commit to solve README alerts and azure-dev.yaml. As I said this an azd multi-platform repo so the validation tool is not going to find infra folder in the root or validate azd up from there ( but it perfectly works if cd in app service, aks, aca folder).

dantelmomsft commented 1 month ago

closing this issue as this should be cleared now with the new commit. if further changes are required to get the repo published in the gallery, we can reopen

anfibiacreativa commented 1 month ago

this an azd multi-platform repo so the validation tool is not going to find infra folder in the root or validate azd up from there ( but it perfectly works if cd in app service, aks, aca folder).

I see. We need to maybe find a pattern to identify this type of structure. Is the deploy folder a convention? If it isn't, we need to work together to find and establish a pattern that would help us define the logical path cc @hund030

anfibiacreativa commented 1 month ago

However, the workflows, are not in place. I was going to be opening a PR for microsoft devops action, but, considering the multiple paths, we may need to think about this together with the above.