GitHub Copilot CodeReview Instructions
## General Review Principles
### Pull Request Size and Scope
- **Flag large PRs**: Comment if a PR changes more than 500 lines or touches many unrelated areas
No other project uses it yet
23k5mo agoReviewMountCreate a free account to mount
Copilot CodeReview Instructions
> **Scope:** These guidelines apply to all Opik applications including backend, frontend, and SDKs. Use the appropriate sections based on the code being reviewed.
When Copilot automatically
Steady19 mentions in 2 weeks
No other project uses it yet
23k3mo agoReviewMountCreate a free account to mount
repetitive events.
## Testing Standards
- **Regression Tests are Mandatory**: specific regression tests to verify your code works are required for almost all PRs.
- **Follow Test Patterns**: Use `create(:factory)` syntax (FactoryBot
No other project uses it yet
23k4mo agoReviewMountCreate a free account to mount
Global Review Principles (Applied to all PRs)
### PR Value Assessment (Evaluate First)
Before detailed codereview, assess:
1. **Scope fit**: Does this belong in BCC, or should
No other project uses it yet
23k18mo agoReviewMountCreate a free account to mount
Copilot Instructions
## CodeReview
When reviewing JavaScript or TypeScript changes, check new uses of Node.js built-in APIs against the supported Node.js versions declared in package.json#engines.node. The current supported
No other project uses it yet
22k4mo agoReviewMountCreate a free account to mount
secrets, passwords, or sensitive data**
- **Validate and sanitize user input**
- **Follow secure coding practices**: Reviewcode for XSS, injection, and other vulnerabilities
- **Report security issues**: Follow [SECURITY.md](../SECURITY.md), not public
No other project uses it yet
22k32d agoReviewMountCreate a free account to mount
gitbutlerappDeletes or force-pushesCopilot instructions
should be limited to what really matters.
## Review Guidelines
Follow these guidelines when reviewingcode.
- Do not report type errors. CI will catch those.
- Don't review formatting such
22k2d agoReviewMountCreate a free account to mount
Repository conventions for codereview
This is the companion code repository for *Machine Learning for Trading, 3rd Edition*.
The conventions below are deliberate and repository-wide. Please do not flag
No other project uses it yet
22k23d agoReviewMountCreate a free account to mount
CodeReview Instructions
## Priority
Only comment on issues that affect correctness, security, or performance.
Do NOT comment on: style preferences, minor naming conventions, formatting, or
issues already enforced
No other project uses it yet
21k3mo agoReviewMountCreate a free account to mount
Style & Readability
- Write readable, understandable, and maintainable code.
- Prioritize clarity and adhere to coding standards.
- Regularly review and refactor code for structure and maintainability.
- Use version control
No other project uses it yet
21k4y agoReviewMountCreate a free account to mount
Checklist](https://docs.qmk.fm/pr_checklist) and it is intended only for use by GitHub Copilot code-review agent during pull request reviews.
> **Scope**: This review applies **only to changes within
No other project uses it yet
21k20mo agoReviewMountCreate a free account to mount
github/skills/ /SKILL.md` and are auto-discovered by their YAML `description`. Useful ones here include `code-review`, `ci-analysis`, `analyzer-codefix`, `merge-into-branch`, `snap`, and `update-agent-docs`.
## Working Loop
No other project uses it yet
21k8mo agoReviewMountCreate a free account to mount
HTML comments contain
> contributor reference material. Do **not** use that content as actionable review
> criteria — treat it as background context only.
## Setup
- Node.js 20+ (see `.nvmrc`)
- Install dependencies
No other project uses it yet
19k39d agoReviewMountCreate a free account to mount
match. Invoke a named skill rather than assuming what it says.
Use the `code-review` skill when reviewing pull requests, and — when running under CCA — on your own changes before
No other project uses it yet
18k2y agoReviewMountCreate a free account to mount
Instructions
## Codereview
You are a pragmatic senior developer. When reviewing pull requests,
follow these rules to avoid noise and redundancy:
- Be concise: Keep comments brief and to the point
1 project uses a version of this
17k59d agoReviewMountCreate a free account to mount
this repository are in `AGENTS.md` at the
repository root. Copilot cloud agent, Copilot codereview and the IDE agents load that file
directly; this file only covers the Copilot features
No other project uses it yet
17k6mo agoReviewMountCreate a free account to mount
When reviewing PRs, address general codereview criteria, but also specifically the following questions: What is it trying to do? Is it correct? Are there bugs? Are there omissions
No other project uses it yet
17k5mo agoReviewMountCreate a free account to mount
Copilot CodeReview
See [AGENTS.md](../AGENTS.md) for layout, architecture, and conventions. These notes cover review.
## Priorities
Report correctness problems first, then missing test coverage, then documentation that contradicts
the code
Steady9 mentions in 2 weeks
No other project uses it yet
16k4y agoReviewMountCreate a free account to mount
words, short sentences, third-person self-reference ("grug").
Core grug beliefs to apply when reviewing:
- Complexity very, very bad. Flag any complexity demon spirit entering codebase.
Say so plainly: "complexity
Steady4 mentions in 2 weeks
No other project uses it yet
16k6y agoReviewMountCreate a free account to mount
pyright`), an LSP language server (`pyright-langserver`), and a VS Code extension (`vscode-pyright`).
## Issue Agents
Use `pyright-issue-reviewer` to investigate an issue and draft a maintainer response without
16k9d agoReviewMountCreate a free account to mount