channels or window lifecycle is ambiguous
- Acknowledge good patterns when you see them
## Testing Standards
- Changes to `src/utils/reader/` (live iframe rendering) require manual verification for layout regressions
- New IPC handlers
No other project uses it yet
29k31d agoReviewMountCreate a free account to mount
files listed in its routing table. Read those, not this file, for workflow,
build, test, and scope rules.
The rules most often missed in review:
- A PR must close
1 project uses a version of this
29k7mo agoReviewMountCreate a free account to mount
src/PowerShell`) - Microsoft.WinGet.Client and Microsoft.WinGet.Configuration cmdlets
- **Configuration system** - DSC-based system configuration using WinGet
## Building, Testing, and Running
### Initial Setup
Use a configuration file in `.config` as in `winget configure .config/configuration.winget
No other project uses it yet
27k22d agoReviewMountCreate a free account to mount
lint:advisory` (includes warnings omitted by the normal quiet lint commands)
- **Unit tests**: `npm run test:unit`
## Code review
When reviewing a PR or reviewing changes on the current branch
Steady3 mentions in 2 weeks
No other project uses it yet
26k4y agoReviewMountCreate a free account to mount
operator/ # Operator proto
placement/ # Placement service proto
scheduler/ # Scheduler service proto
sentry/ # Sentry proto
tests/
integration/ # Integration tests (run against real daprd binary)
e2e/ # End-to-end tests (require Kubernetes
No other project uses it yet
26k6mo agoReviewMountCreate a free account to mount
development takes place in the `internal` directory, and most behaviors can be tested via compiler tests.
Most development on the codebase is in Go.
Standard Go commands and practices apply
26k9mo agoReviewMountCreate a free account to mount
last resort** and only for critical hotfixes
This ensures changes are properly tested in beta releases with appropriate version bumps before being merged to the main release branch.
## Beta Branch
No other project uses it yet
26k4mo agoReviewMountCreate a free account to mount
optional features
- Global variables not in `lv_global_t` — must use `LV_GLOBAL_DEFAULT()`
- Test headers included in production source (`#include "lv_test_..."` in `src/`)
- Missing `LV_CHECK
No other project uses it yet
25k18d agoReviewMountCreate a free account to mount
manually fix the files indicated by the linter, then re-run the command.
- Testing:
- Run package tests where they live:
- `pnpm --filter @blue-frontier/dev-sidecar test`
- `pnpm --filter @blue-frontier
No other project uses it yet
25k23d agoReviewMountCreate a free account to mount
acceptable. Report the gap if its consequence matters once the change is released.
- A test that asserts a behavior shows that the author intended the behavior, not that the behavior
No other project uses it yet
24k4mo agoReviewMountCreate a free account to mount
them when supported by actual compiler diagnostics.
- Treat files inside any `fixtures` directories as test data, not production code.
- Fixture files may intentionally contain buggy, unsafe, or syntactically invalid code
No other project uses it yet
23k13d agoReviewMountCreate a free account to mount
versioning policy**: Point to the requirement of at least one version notice before removal
### Testing and Documentation
- **Verify test coverage**: Ensure new functionality includes appropriate tests
- **Check for test removal
No other project uses it yet
23k5mo agoReviewMountCreate a free account to mount