Commits
8. Push to your fork
9. Create Pull Request targeting the `next` branch
### CodeReview
- Be responsive to feedback
- Keep discussions constructive and professional
- Make requested changes promptly
2.4k13mo agoReviewMountCreate a free account to mount
indexes all project docs: implementation guidance, conventions, architecture, and the release-review checklist.
Before writing any code, read and follow the standing rules: [.github/instructions/implementation.instructions.md](instructions/implementation.instructions.md) and [.github/instructions/project-notes.instructions.md](instructions/project-notes.instructions.md). Other docs
Steady8 mentions in 2 weeks
No other project uses it yet
2.4k4mo agoReviewMountCreate a free account to mount
Contribution Guidelines: Enhanced
Follow these mandatory guidelines when contributing code, documentation, and translations to this repository. All contributions will be reviewed for compliance.
---
## General Project Overview
* **Project Scope:** This
Steady7 mentions in 2 weeks
No other project uses it yet
2.4k2y agoReviewMountCreate a free account to mount
full checks from `AGENTS.md` for
code changes. Do not weaken a lint or test to make a change pass.
## Review communication
After the scope and evidence checks, lead with concrete
2.4k39d agoReviewMountCreate a free account to mount
sources on `master`; for new pipeline sources use the release branch guide above.
## Things reviewers consistently reject
- Hardcoded dates or weekday rules — sources must fetch from a real upstream
2.3k4mo agoReviewMountCreate a free account to mount
implementation-and-provenance)
required by `AGENTS.md`. Apply it to delegated agents and their tools.
## CodeReview Checklist
When reviewing PRs, verify:
- [ ] **Documentation**: All exported APIs have rustdoc comments (missing docs
2.3k9mo agoReviewMountCreate a free account to mount
examples, tests, build or docs changes, and downstream APIs.
4. Analyze the code against the critical review pillars.
5. Compare the change against nearby code and MFEM patterns; flag unmotivated
2.3k10d agoReviewMountCreate a free account to mount
claude/skills/build-and-verify/SKILL.md` for build, compile, run, and test tasks
- Use `.claude/skills/cpp-coding/SKILL.md` for C++ code changes and reviews
- Use `.claude/skills/sql-usage/SKILL.md` for streaming SQL work
- Use `.claude/skills/review/SKILL.md` for PR or diff review
2.3k7mo agoReviewMountCreate a free account to mount
GitHub Copilot CodeReview Instructions
## Review Philosophy
- Only comment when you have HIGH CONFIDENCE (>80%) that an issue exists
- Be concise: one sentence per comment when possible
- Focus on actionable
2.3k3mo agoReviewMountCreate a free account to mount
WeRead (微信读书) book metadata, highlights, notes, and reviews into Markdown files in an Obsidian vault.
- The plugin supports login via WeChat QR code, cookie management, custom templates (Nunjucks), and flexible
2.3k3y agoReviewMountCreate a free account to mount
code, comments, and documentation.
## AI policy
This project follows the [Open Home Foundation AI Policy](../AI_POLICY.md). Autonomous contributions are not accepted: a human must review, understand, and be able
2.3k17d agoReviewMountCreate a free account to mount
part of the full suite
4. Coverage reporting is available through make targets
## CodeReview Rule
你是一个高级代码审查助手,审查代码时要从QA角度仔细检查问题,以批判的眼光看待代码,以发现潜在问题为目的。
为了避免得到假阳性的检查结果,请注意:
- 分析具体代码片段时要包含足够上下文,不要仅基于局部信息做出判断。
- 避免基于记忆进行代码分析,必须基于实际查看的代码。
- 在指出问题前,先理解业务逻辑的完整流程,考虑代码设计的合理性和必要性。
针对每个有变更的文件及其差异块
2.2k4mo agoReviewMountCreate a free account to mount
branches.
Breaking change(s):
- [Description of the breaking change]
```
## Changelog Validation
**When reviewing PRs, check if code changes have a changelog fragment under `changelog.d/`.**
Changelog entries are added as fragment
No other project uses it yet
2.2k4y agoReviewMountCreate a free account to mount
ERROR_FILE_TOO_LARGE)` |
| `HRESULT_E_CANNOT_MAKE` | `HRESULT_FROM_WIN32(ERROR_CANNOT_MAKE)` |
## CodeReview Instructions
When reviewingcode, focus on the following aspects:
- Adherence to coding standards defined
2.2k4mo agoReviewMountCreate a free account to mount
clear regression risk**.
- If you are not confident it is must-fix, **stay silent**.
#### Review priorities (in order)
1. **Behavior safety**
- For valid inputs, changes should
2.2k3mo agoReviewMountCreate a free account to mount
initialize `TaskEnvironment` to `Fallback` when implementing the interface directly so direct instantiation also works.
- Review helpers, exception paths, and static state for thread safety. Do not mutate shared process state
2.1k47d agoReviewMountCreate a free account to mount
Copilot CodeReview
- Every image change in a contributor PR requires a manifest bump. For breaking changes such as a base OS change, increment the major component and reset minor
2.1k20mo agoReviewMountCreate a free account to mount
When performing a codereview on a pull request, verify the following things:
* there is no merge commit among the commits of the pull request,
* the commit messages use American
2.1k6mo agoReviewMountCreate a free account to mount
Encoder Source Location
When a task requires viewing, searching, reviewing, editing, or testing Encoder code, use the
Encoder source at `D:\git\htdocs\AVideo\.compose\encoder\`. Treat
2.1k23mo agoReviewMountCreate a free account to mount