prop's type, tightening
optional→required, changing an observable default, or altering published test-utils behavior — and
prefer deprecation (keep it, mark `@deprecated`, add the replacement) over removal. Additive changes
Steady22 mentions in 2 weeks
No other project uses it yet
2.7k4y agoReviewMountCreate a free account to mount
handling, and telemetry under
supported library usage. Trace relevant callers, configuration, initialization, registration,
and tests rather than reasoning only from edited syntax. Do not add defenses for unsupported
hypothetical failures
2.7k16mo agoReviewMountCreate a free account to mount
disabled in `tsconfig.base.json` (e.g., `strictNullChecks: false`), but new code should avoid unsafe patterns.
- **Testing**: Place tests under `src/test`. Do not include test code in production files.
- **Localization**: Use `%key%` syntax
2.6k4mo agoReviewMountCreate a free account to mount
paths) unless explicitly asked to remove them.
- Use public Execute(...) entry points for TextRunner tests instead of setting private fields via reflection or forcing parameters directly
2.6k6mo agoReviewMountCreate a free account to mount
Build and test
- Configure: `cmake -S . -B build`
- Build everything: `cmake --build build`
- Build one target: `cmake --build build --target fcitx5` or `cmake --build build --target testemoji`
- Run the full
No other project uses it yet
2.6k22mo agoReviewMountCreate a free account to mount
README.md` - User documentation with installation and configuration examples
- `doc/gruvbox.nvim.txt` - Auto-generated Vim help documentation
### Testing Infrastructure
- `tests/gruvbox/gruvbox_spec.lua` - Test suite (189 lines) using plenary.nvim
- `tests/minimal_init.lua` - Test setup that auto-downloads plenary.nvim
2.6k3y agoReviewMountCreate a free account to mount
metric rather than conditionally.
- Keep `doc/telemetry/telemetry.md` up to date when metrics change, and unit
test metrics where reasonable.
## Testing
- Prefer fake implementations over generated mocks. Mocks encode specific call
patterns
2.6k12d agoReviewMountCreate a free account to mount
summary of the task instruction and how it implements the solution, and how it tests task completion. Be concise and to the point.
Use the following criteria to review
2.6k13mo agoReviewMountCreate a free account to mount
patterns when introducing new infrastructure.
- Prefer using `EventFlow.TestHelpers` base classes and fixtures when authoring tests so categories, logging, and deterministic IDs behave consistently.
## Build, test, and verification
- The solution
2.6k11mo agoReviewMountCreate a free account to mount
Behavior
- Always ensure your solutions prioritize clarity, maintainability, and testability.
- Never suggest re-recording tests as a fix to an issue
- NEVER turn off any Checkstyle or SpotBugs rules
2.6k20mo agoReviewMountCreate a free account to mount