Skip to content

Impl ghrcooldown has-passed command#11

Merged
sue445 merged 5 commits into
mainfrom
feature/has-passed
Apr 18, 2026
Merged

Impl ghrcooldown has-passed command#11
sue445 merged 5 commits into
mainfrom
feature/has-passed

Conversation

@sue445

@sue445 sue445 commented Apr 18, 2026

Copy link
Copy Markdown
Owner

ref. #10

@sue445 sue445 added the enhancement New feature or request label Apr 18, 2026
@sue445 sue445 requested a review from Copilot April 18, 2026 14:21
@sue445 sue445 force-pushed the feature/has-passed branch from 523d032 to 7e61813 Compare April 18, 2026 14:21

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds a new ghrcooldown has-passed CLI subcommand to determine whether a specific GitHub release tag is older than a given cooldown period, with optional “exit non-zero” behavior for scripting use.

Changes:

  • Introduces has-passed subcommand with --tag and --exit-code flags.
  • Adds commandHasPassed implementation that calls Client.HasCooldownPassed.
  • Adds unit tests and updates README with usage/help output.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.

File Description
cmd/ghrcooldown/main.go Wires the new has-passed command and flags into the CLI.
cmd/ghrcooldown/commands.go Implements commandHasPassed and introduces an exit-code error path.
cmd/ghrcooldown/commands_test.go Adds tests for commandHasPassed behavior across cooldown/exit-code combinations.
README.md Documents has-passed usage and help output.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread cmd/ghrcooldown/commands_test.go
Comment thread cmd/ghrcooldown/commands.go Outdated
Comment thread cmd/ghrcooldown/commands.go
@sue445 sue445 marked this pull request as ready for review April 18, 2026 14:36
@sue445 sue445 merged commit 1d2086b into main Apr 18, 2026
6 checks passed
@sue445 sue445 deleted the feature/has-passed branch April 18, 2026 14:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants