Skip to content

Rename -days flag to -sync-lookback-days#30

Merged
justinabrahms merged 1 commit into
mainfrom
work/witty-panda
Jan 22, 2026
Merged

Rename -days flag to -sync-lookback-days#30
justinabrahms merged 1 commit into
mainfrom
work/witty-panda

Conversation

@justinabrahms
Copy link
Copy Markdown
Owner

Summary

Renamed the -days flag to -sync-lookback-days to clarify its purpose. This flag controls how far back the GitHub API sync fetches data, not what appears in the report. The -since flag independently controls report filtering.

Changes

  • Renamed flag from -days to -sync-lookback-days
  • Updated flag description: "How far back to fetch GitHub data (1-365 days, doesn't affect report filtering)"
  • Updated Config struct comment for clarity
  • Updated all test references (3 test cases)
  • Updated README.md documentation with new flag name and clearer example comment
  • Kept default value at 30 days

Testing

  • All existing tests pass
  • Build succeeds
  • Help output shows new flag correctly

🤖 Generated with Claude Code

Clarifies that this flag controls how far back the GitHub API sync
fetches, not what appears in the report. The -since flag controls
report filtering independently.

- Renamed flag from -days to -sync-lookback-days
- Updated flag description for clarity
- Updated all test references
- Updated README.md documentation
- Kept default value at 30 days

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@justinabrahms justinabrahms added the multiclaude PR created by multiclaude worker label Jan 22, 2026
@justinabrahms justinabrahms merged commit fb1e9e5 into main Jan 22, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

multiclaude PR created by multiclaude worker

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant