Skip to content

Add copilot instruction files for sample conventions#4

Closed
diberry wants to merge 2 commits into
article2/select-algorithmfrom
ci/copilot-instructions
Closed

Add copilot instruction files for sample conventions#4
diberry wants to merge 2 commits into
article2/select-algorithmfrom
ci/copilot-instructions

Conversation

@diberry

@diberry diberry commented Apr 30, 2026

Copy link
Copy Markdown
Owner

Summary

Adds copilot instruction files documenting how samples should be built in this repo.

Files

  • .github/copilot-instructions.md\ - Main shared conventions (naming, env vars, lifecycle)
  • .github/copilot-instructions-{language}.md\ - Language-specific patterns (bulk insert, env loading)

Depends on

diberry and others added 2 commits April 30, 2026 07:51
…rop at end

All 10 sample directories now follow the same pattern:
- START: conditionally drop collection only if it exists
- END: always drop collection for cleanup (in finally/defer block)

Languages updated: TypeScript, Python, Go, Java, .NET

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Main instructions: naming, env vars, collection lifecycle, bulk insert
- Language-specific files for TypeScript, Python, Go, Java, .NET
- Documents how samples should be built for CI compatibility

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@diberry

diberry commented Apr 30, 2026

Copy link
Copy Markdown
Owner Author

Retargeting to upstream Azure-Samples/documentdb-samples

@diberry diberry closed this Apr 30, 2026
diberry added a commit that referenced this pull request May 20, 2026
- Create ai/includes/choosing-algorithm.md with enhanced content
- Add quick-reference decision table (IVF/DiskANN/HNSW by scenario)
- Elevate DiskANN-as-default recommendation with IMPORTANT callout
- Add operational benefits: easier backups, faster recovery
- Add dimension future-proofing context (models evolving past 8K)
- Replace duplicated sections in all 5 quickstarts with include ref
- Addresses Khelan Modi feedback points #3 and #4

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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.

1 participant