avoidable
Python-level calls; never trade documented subclass hooks or semantics for speed.
## CodeReview
- Reviews **amend `.github/copilot-review.md` in place** (header: date, commit, CI
status, coverage, one *Verified* paragraph); never
2.8k3mo agoReviewMountCreate a free account to mount
GitHub Copilot Instructions
This file configures GitHub Copilot (code suggestions and PR reviews) to follow the coding
standards enforced by the project's pre-commit hooks (`.pre-commit-config.yaml`) and
tool configuration
2.7k13d agoReviewMountCreate a free account to mount
when a review claim depends on browser rendering, interaction,
responsive layout, hydration, or accessibility. When an unstable custom lint
fails, evaluate the rule as well as the code before recommending
2.7k3mo agoReviewMountCreate a free account to mount
Guidelines
* Use GitHub Copilot to assist with code completion, documentation, and generating boilerplate code.
* Always review and test the code suggested by GitHub Copilot to ensure it meets the project
2.7k23mo agoReviewMountCreate a free account to mount
Coding Guidelines
## Introduction
These are VS Codecoding guidelines. Please also review our [Source Code Organisation](https://github.com/microsoft/vscode/wiki/Source-Code-Organization) page.
## Indentation
We use tabs, not spaces.
## Naming Conventions
* Use PascalCase
2.7k4y agoReviewMountCreate a free account to mount
OpenTelemetry Java Instrumentation codereviewReview the pull request's changed behavior, not just the edited syntax. Trace relevant callers,
configuration, lifecycle, and tests before drawing a conclusion. Report
2.7k15mo agoReviewMountCreate a free account to mount
tests task completion. Be concise and to the point.
Use the following criteria to review the task:
1. **Behavior in Task Description**: Check whether all behavior tested in the test
2.6k13mo agoReviewMountCreate a free account to mount
processes, the SPIRE Server and the SPIRE Agent, plus a plugin system.
When reviewing or generating code, weigh the concerns covered below. All of
them matter, and more than
2.6k2d agoReviewMountCreate a free account to mount
those when suggesting changes.
## AI tooling lives elsewhere
Reusable AI workflows for this project (codereview, werk authoring,
Gerrit/Jenkins/Jira/crash-report helpers, backports, etc.) are maintained
as Claude Code plugins
Steady7 mentions in 2 weeks
No other project uses it yet
2.6k5mo agoReviewMountCreate a free account to mount
NEVER turn off any Checkstyle or SpotBugs rules to resolve linting issues.
- Always review your own code for consistency, maintainability, and testability
- Always ask how to verify that your changes
2.6k20mo agoReviewMountCreate a free account to mount
Copilot instructions for this repository
## High level guidance
* Review the `CONTRIBUTING.md` file for instructions to build and test the software.
* Set the `NBGV_GitEngine` environment variable to `Disabled` before running
Steady4 mentions in 2 weeks
No other project uses it yet
2.5k6mo agoReviewMountCreate a free account to mount
Copilot instructions
When generating code or reviewing pull requests for this repository, use
`CONTRIBUTING.md` as the source of truth for contributor requirements.
For pull request reviews, check the changed files
2.4k5d agoReviewMountCreate a free account to mount
block, pass state through `PostProcessingArgs`.
## Code Formatting & Style
These formatting rules are enforced during codereview. Agents must apply them to all generated or modified PowerShell code:
1. **No trailing
2.4k2mo agoReviewMountCreate a free account to mount
patterns over custom non-standard lifecycle hooks, especially for IDisposable/disposal implementations, to keep codereview and commit ownership clear.
- Avoid changes that cause `DialogManager` static initialization when the client/server process
No other project uses it yet
2.4k16mo agoReviewMountCreate a free account to mount
analysis and fix all warnings
- Follow the existing code structure and patterns
- Check that your changes don't break compatibility
- Review security implications of your changes
- Contributors must agree
2.4k5d agoReviewMountCreate a free account to mount
existing pattern does so via an abstraction.
- Don't capture `Console` I/O in library code (leave printing/logging to samples/apps).
- Don't add hidden thread‑static caches or background threads without
2.4k6y agoReviewMountCreate a free account to mount
relevant issues or discussions
- Any additional context that may be helpful for reviewers
## CodeReview Guidelines
When performing a codereview, please follow these guidelines:
- If there are changes that
2.4k16mo agoReviewMountCreate a free account to mount