systematically check all methods, relied on intuition
- **Improvement**: Added "Always Examine Actual Source Code" as critical prerequisite with grep workflow
- **Issue**: UrlHelper::url_to_youtube_vid() test missed despite being
87413mo agoReviewMountCreate a free account to mount
tests # Run all tests with coverage
lint # Check code with ruff
format # Format code with ruff
typecheck # Run mypy type checker
build # Build package distributions
```
**Python-only alternative
87059d agoReviewMountCreate a free account to mount
files directly.
- Follow the platform-specific rule and test layout documented in `docs\RuleContributions.md`.
- Review generated SARIF baselines when rule applicability, messages, or output changes.
## Parser and driver changes
- Treat
86841d agoReviewMountCreate a free account to mount
FROM_WIN32(ERROR_INVALID_NAME)` |
| `E_BOUNDS` | `0x8000000BL` (conditionally defined if not already present) |
## CodeReview Instructions
When reviewingcode, focus on the following aspects:
- Adherence to coding standards defined
8614mo agoReviewMountCreate a free account to mount
When reviewingcode:
- Do NOT generate pull request overviews.
- Do NOT generate explainers.
- Do NOT summarize what a diff chunk does.
- Only point out issues that are of medium severity
85820mo agoReviewMountCreate a free account to mount
src/aiperf/plugin/enums.py` by `make generate-all-plugin-files`. Never hand-edit `src/aiperf/plugin/enums.py`.
- **Operator-namespace fallback** — code that needs the chart-default operator namespace MUST import `DEFAULT_OPERATOR_NAMESPACE` from `aiperf.kubernetes.constants`, never
83952d agoReviewMountCreate a free account to mount
OpenTelemetry Proto Repository Instructions
First-pass PR review rules for OpenTelemetry Protobuf definitions.
**Prefer silence over uncertainty.** Only flag substantive issues on changed lines.
Skip stylistic preferences not listed
8264mo 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
8246d agoReviewMountCreate a free account to mount
Docker; do not introduce direct process execution
## Auth Review
`floci-az.auth.mode` (`dev` or `strict`) is not read by any code today: every request is handled as
in `dev`, and `SharedKeyAuthVerifier` accepts
81421d agoReviewMountCreate a free account to mount
description.
3. **Preview-feature carve-out.** If the change is to a preview-only
code path *and* it could affect default-config semantics for any
customer in the next
7986mo agoReviewMountCreate a free account to mount
background on an unfamiliar subsystem — call for dedicated research
rather than jumping straight into code.
- Before starting new research, check whether a prior-art survey or design note already exists
79513mo agoReviewMountCreate a free account to mount
Provide clear PR description explaining the changes
5. Be responsive to review feedback
### CodeReview Process
- PRs require passing CI checks before merge
- Maintainer review is required
- Keep PRs focused
7864mo agoReviewMountCreate a free account to mount
General CodeReview Standards
## Purpose
These instructions guide GitHub Copilot codereview across all files in this repository.
## About Open RV
Open RV is an open-source, hardware-accelerated image
78316mo agoReviewMountCreate a free account to mount
CodeReview Instructions: `balance` (Python)
Review instructions for the **balance** Python package (weighting and balancing utilities for correcting bias in tabular datasets). Prioritize correctness, statistical validity, reproducibility, and backward compatibility
7776mo agoReviewMountCreate a free account to mount