Skip to content

feat: show estimate in issues list, search, and read output#92

Merged
flipbit03 merged 4 commits intoflipbit03:mainfrom
lightstrikelabs:feat/issues-estimate-output
Mar 6, 2026
Merged

feat: show estimate in issues list, search, and read output#92
flipbit03 merged 4 commits intoflipbit03:mainfrom
lightstrikelabs:feat/issues-estimate-output

Conversation

@lightstrike
Copy link
Copy Markdown
Contributor

@lightstrike lightstrike commented Feb 20, 2026

Summary

  • Add estimate: Option<f64> to IssueSummary, SearchSummary, and IssueDetail types
  • Add estimate column to IssueRow table output
  • Format estimate as integer when whole number (e.g. "3" not "3.0"), empty string when None
  • Estimate now appears in issues list, issues search, and issues read output

Test plan

  • cargo clippy --workspace -- -D warnings clean
  • cargo fmt --check clean
  • cargo test --workspace — all 64 offline tests pass
  • Smoke test: lineark issues list --limit 5 — estimate column visible
  • Smoke test: lineark issues search "test" -l 5 — estimate column visible
  • Smoke test: lineark issues read ENG-xxx — estimate field in detail output

Copy link
Copy Markdown
Owner

@flipbit03 flipbit03 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, from a functionality standpoint, but we should not delete these comments from unrelated pieces. Issue documented in #97

@flipbit03 flipbit03 added the safe-to-test Maintainer-approved: run online tests with API secrets label Feb 24, 2026
@github-actions github-actions bot removed the safe-to-test Maintainer-approved: run online tests with API secrets label Feb 24, 2026
@lightstrike lightstrike force-pushed the feat/issues-estimate-output branch from 4ad3a47 to 5ff5cb1 Compare February 28, 2026 21:53
@lightstrike
Copy link
Copy Markdown
Contributor Author

@flipbit03 Ready for review again — rebased onto main and reverted the unrelated comment deletions. All CI checks passing.

@flipbit03 flipbit03 force-pushed the feat/issues-estimate-output branch from 5ff5cb1 to b82a840 Compare March 6, 2026 18:41
@flipbit03 flipbit03 force-pushed the feat/issues-estimate-output branch from b82a840 to 6a120ac Compare March 6, 2026 18:51
Copy link
Copy Markdown
Owner

@flipbit03 flipbit03 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@flipbit03 flipbit03 added the safe-to-test Maintainer-approved: run online tests with API secrets label Mar 6, 2026
@github-actions github-actions bot removed the safe-to-test Maintainer-approved: run online tests with API secrets label Mar 6, 2026
@flipbit03 flipbit03 merged commit f8ee6d2 into flipbit03:main Mar 6, 2026
9 checks passed
@flipbit03 flipbit03 deleted the feat/issues-estimate-output branch March 6, 2026 19:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants