Skip to content
Merged
Prev Previous commit
Next Next commit
Address code review feedback for consistency
Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
  • Loading branch information
Copilot and Malcolmnixon committed Feb 10, 2026
commit fea0424f2d05c3bfdcecb089b76c286770d5a87f
2 changes: 1 addition & 1 deletion .github/agents/software-developer.md
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@ var results = ProcessFile(options.InputFile);
### BuildMark-Specific Rules

- **XML Docs**: On ALL members (public/internal/private) with spaces after `///`
- Follow standard XML indentation rules with four-space indentation
- Follow standard XML documentation formatting with 4-space indentation
- **Errors**: `ArgumentException` for parsing, `InvalidOperationException` for runtime issues
- **Namespace**: File-scoped namespaces only
- **Using Statements**: Top of file only
Expand Down
3 changes: 2 additions & 1 deletion AGENTS.md
Original file line number Diff line number Diff line change
Expand Up @@ -23,9 +23,10 @@ Project-specific guidance for agents working on BuildMark - a .NET CLI tool for

## Requirements

- Link ALL requirements to tests (prefer `BuildMark_*` self-validation tests)
- Link ALL requirements to tests (prefer `BuildMark_*` self-validation tests for command-line behavior)
- Enforced in CI: `dotnet reqstream --requirements requirements.yaml --tests "test-results/**/*.trx" --enforce`
- When adding features: add requirement + link to test
- See Requirements Agent for detailed test coverage strategy

## Testing

Expand Down