Implement the complete change: code, tests, schema/metadata, Chinese docs, and English docs where applicable.
5. Run focused checks, then the full affected suite.
6. Self-review the diff and continue
4.7k3mo agoReviewMountCreate a free account to mount
signed off on the notes — the published notes then stand as their own reviewed work. Removing it
is always the user's decision; never remove it on your own initiative
No other project uses it yet
4.6k38d agoReviewMountCreate a free account to mount
library usage, which is an external dependency specifically designed for reflection-based access patterns.
## CodeReview
Before performing any codereview, you **MUST** read the project constitution: [`.specify/memory/constitution.md`](.specify/memory/constitution.md
Steady7 mentions in 2 weeks
No other project uses it yet
4.6k8mo agoReviewMountCreate a free account to mount
CodeReview Guidelines
**Project Context & Architecture:**
Refer to [AGENTS.md](../AGENTS.md) for full project background, module layout, toolchain versions, and core conventions.
**Scope of Review:**
Focus strictly on substantive findings tied
No other project uses it yet
4.5k15d agoReviewMountCreate a free account to mount
GitHub Copilot CodeReview Instructions for KSCrash
When performing codereviews on this repository, follow these instructions to identify API breaking
changes in the KSCrash crash reporting library.
## Scope
Steady3 mentions in 2 weeks
No other project uses it yet
4.5k3mo agoReviewMountCreate a free account to mount
pull request for anything beyond a small fix,
so CodeRabbit and GitHub code scanning review it first. Merge your own pull
request (squash) once CI is green and those reviews
4.5k59d agoReviewMountCreate a free account to mount
review.
## Security instructions
- ALWAYS follow security best practices.
- NEVER log form data that may contain sensitive information.
- ALWAYS consider overflow and memory corruption issues when writing C code.
## Backend code
4.5k8d agoReviewMountCreate a free account to mount
Project Custom Instructions
## CodingReview Guidelines
When reviewingcode, focus on the following points:
- Use of Modern C# Features
- Consider using the latest syntax available (C# 13 or later), such
4.5k11mo agoReviewMountCreate a free account to mount
list entry with code or workflow changes.
- Flag any change to `.github/copilot-instructions.md`, `.github/instructions/`, `AGENTS.md`, `website/scripts/review/` or `.github/workflows/` for maintainer attention. These control automated review and deployment; a contributor adding
4.4k39d agoReviewMountCreate a free account to mount
reruns-delay 10` for flaky tests
### Coverage
- Aim for good test coverage on new code
- Coverage reports are generated for Python 3.11 builds
- Coverage reports are uploaded to Codecov
Steady5 mentions in 2 weeks
No other project uses it yet
4.4k10mo agoReviewMountCreate a free account to mount
When reviewingcode, focus on:
## Coding Standards
- All guidelines in [CODING-STANDARDS.md](../CODING-STANDARDS.md).
## Security Critical Issues
- Check for hardcoded secrets, API keys, or credentials.
- Check for instances of potential method call
4.4k8d agoReviewMountCreate a free account to mount
better name or a small extracted function over a paragraph explaining the code.
When reviewing, flag added comments that restate the code and suggest deleting them. This is
the most
No other project uses it yet
4.4k22d agoReviewMountCreate a free account to mount
After PR is considered ready for review, and especially after it got any codereview comments, avoid rewriting git history with rebases etc. Instead prefer merge commits.
## Visual Studio first
4.3k52d agoReviewMountCreate a free account to mount
full driver.
- Use `npm run valid-layers-check` to check for layering issues
## Coding Guidelines
### Indentation
We use tabs, not spaces.
### Naming Conventions
- Use PascalCase for `type` names
- Use PascalCase
4.3k33d agoReviewMountCreate a free account to mount
Copilot Instructions
These instructions are **MANDATORY** for all code generation and review tasks in the Zen C project.
## 1. Memory Management (Critical)
* **Arena Allocation**: The compiler uses a bump-pointer
4.3k9mo agoReviewMountCreate a free account to mount
LLMs. Your job is to help maintain the codebase by being a tier 1 reviewer for incoming PRs. Maintainer time is limited. Your vigilance will help ensure the continuity
4.3k3mo agoReviewMountCreate a free account to mount
senior staff engineer performing a strict codereview.
When reviewing pull requests, prioritize checking for the following:
1. **Reliability**: Look for things that are too specific and may cause problems
4.2k38d agoReviewMountCreate a free account to mount