-
Notifications
You must be signed in to change notification settings - Fork 18
docs: Update CHANGELOG for v0.2.8 #33
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Resolve merge conflicts between Crush support (this branch) and master's changes including CopilotCli support. Note: CopilotCli support is temporarily removed as the harness-barn git branch dependency only includes Crush support. CopilotCli will be re-added once harness-barn includes both harness variants. Changes: - Keep Crush harness support with git branch dependency - Update test infrastructure with master's TestEnvGuard pattern - Add Crush install instructions and display name mappings - Mark Crush CLI tests as ignored (require harness to be installed, as harness-locate doesn't support XDG_CONFIG_HOME override on macOS)
- Update to harness-locate 0.5.0 (git master) which supports both CopilotCli and Crush - Add CopilotCli harness mappings to all relevant files - Add OS-specific installation instructions for Copilot CLI - Add copilot-cli/copilot/ghcp aliases for harness resolution Both harnesses now fully supported with profile management, MCP installation, and TUI.
- Add Copilot CLI back to all harness lists, tables, and error messages - Restore copilot/copilot-cli aliases in all parse_harness_kind() functions - Restore copilot_cli_instructions() and install path mappings - Restore 4 Copilot tests removed in PR #26 - Update harness-barn dependency to neiii fork with both CopilotCli and Crush Fixes: PR #26 accidentally replaced Copilot with Crush instead of adding alongside
- Add Crush CLI harness support (thanks @rari404) - Add Copilot CLI harness support (thanks @kaiiiiiiiii) - Add TUI profile creation improvements (#23) - Document npm installation guidance and README updates
Greptile SummaryThis PR updates the CHANGELOG to document v0.2.8 release notes, which includes Crush CLI harness support, restoration of Copilot CLI support, TUI profile creation improvements (#23), and documentation enhancements. Key Changes:
Issues Found:
Confidence Score: 4/5
Important Files Changed
Sequence DiagramsequenceDiagram
participant Dev as Developer
participant PR as PR #33
participant CL as CHANGELOG.md
participant CT as Cargo.toml
participant RM as README.md
Dev->>PR: Create PR to document v0.2.8
PR->>CL: Add v0.2.8 release notes
CL->>CL: Document Crush CLI support
CL->>CL: Document Copilot CLI restoration
CL->>CL: Document TUI improvements (#23)
CL->>CL: Document README updates
Note over PR,RM: Earlier commits in PR
PR->>RM: Restructure installation section
PR->>RM: Add Crush/Copilot acknowledgments
PR->>CT: Switch to git dependencies
Dev->>CT: Need to bump version 0.2.7→0.2.8
Dev->>RM: Fix typo: intergration→integration
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
18 files reviewed, 2 comments
| The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/), | ||
| and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html). | ||
|
|
||
| ## [0.2.8] - 2026-01-20 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
style: Check that version is bumped to 0.2.8 in Cargo.toml before release (currently still shows 0.2.7).
Prompt To Fix With AI
This is a comment left during a code review.
Path: CHANGELOG.md
Line: 8:8
Comment:
**style:** Check that version is bumped to `0.2.8` in `Cargo.toml` before release (currently still shows `0.2.7`).
How can I resolve this? If you propose a fix, please make it concise.| - Thank you Melvyn for [pointing out my stupidity](https://x.com/melvynxdev/status/2007312037920289275?s=20) | ||
| - Thank you Theo for [giving me a chance to show my work](https://x.com/theo/status/2007255666256425431) | ||
| - Thank you [Kai](https://github.com/kaiiiiiiiii) for the help on GitHub Copilot CLI integration | ||
| - Thank you [rari404](https://github.com/edlsh) for the help on Crush CLI intergration |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
syntax: Typo: "intergration" should be "integration"
| - Thank you [rari404](https://github.com/edlsh) for the help on Crush CLI intergration | |
| - Thank you [rari404](https://github.com/edlsh) for the help on Crush CLI integration |
Prompt To Fix With AI
This is a comment left during a code review.
Path: README.md
Line: 171:171
Comment:
**syntax:** Typo: "intergration" should be "integration"
```suggestion
- Thank you [rari404](https://github.com/edlsh) for the help on Crush CLI integration
```
How can I resolve this? If you propose a fix, please make it concise.
Summary
Updates CHANGELOG.md with all changes since v0.2.7, preparing for v0.2.8 release.
Changes
Added
Documentation
Contributors
Special thanks to:
Ready to merge for v0.2.8 release.