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
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
Steady3 mentions in 2 weeks
No other project uses it yet
2.4k5mo 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
Steady15 mentions in 2 weeks
No other project uses it yet
2.4k2y agoReviewMountCreate a free account to mount
that produced the change.
- Reviewers will reject PRs whose human-facing comments don't read as if a
person wrote them.
### Agent notes
- Prefix a code comment with `Agent Note
2.3k4d 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
No other project uses it yet
2.3k4mo 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
Steady7 mentions in 2 weeks
No other project uses it yet
2.3k3y 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.3k10mo 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.3k12d 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.3k8mo 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
Steady2 mentions in 2 weeks
No other project uses it yet
2.3k3mo 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.3k30d agoReviewMountCreate a free account to mount
part of the full suite
4. Coverage reporting is available through make targets
## CodeReview Rule
你是一个高级代码审查助手,审查代码时要从QA角度仔细检查问题,以批判的眼光看待代码,以发现潜在问题为目的。
为了避免得到假阳性的检查结果,请注意:
- 分析具体代码片段时要包含足够上下文,不要仅基于局部信息做出判断。
- 避免基于记忆进行代码分析,必须基于实际查看的代码。
- 在指出问题前,先理解业务逻辑的完整流程,考虑代码设计的合理性和必要性。
针对每个有变更的文件及其差异块
2.2k5mo 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
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.1k11d 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