[Docs] Add Copilot instructions and pr-finalize skill#1579
Conversation
Adds a comprehensive Copilot instructions file covering: - Build, test, and lint commands (including single-test execution) - Unified single-store architecture with three context types - Step-by-step guide for adding new RESP commands - Custom extensions registration patterns - Test structure with Allure/NUnit requirements - Configuration, coding style, and performance conventions Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Adds a repository-specific Copilot instructions document intended to help AI assistants contribute effectively to Garnet by summarizing build/test commands, core architecture, command implementation workflow, extension points, and key coding/testing conventions.
Changes:
- Introduces
.github/copilot-instructions.mdwith verified build/test/format commands and repo navigation pointers. - Documents the unified single-store architecture (string/object/unified contexts;
ValueIsObjectrecord discrimination). - Provides a step-by-step guide for adding RESP commands and registering custom extensions.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Change XML doc comments from 'required' to 'strongly recommended' with note about .editorconfig suggestion-level severity - Fix duplicated 'Common optional parameters' sentence and double-period typo in server lifecycle bullet - Fix SaveKeyEntryToLock signature: remove non-existent isObject parameter, document PinnedSpanByte key type Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…oft/garnet into badrishc/add-copilot-instructions
badrishc
changed the title
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Adds Copilot instructions and a PR finalization skill to help AI assistants work effectively in this repository.
What's included
Copilot instructions (
.github/copilot-instructions.md)PR finalize skill (
.github/skills/pr-finalize/)Notes