Skip to content

Comments

Update README with duration flag information and example#43

Merged
ldemailly merged 3 commits intomainfrom
ldemailly-patch-2
Sep 11, 2025
Merged

Update README with duration flag information and example#43
ldemailly merged 3 commits intomainfrom
ldemailly-patch-2

Conversation

@ldemailly
Copy link
Member

@ldemailly ldemailly commented Sep 11, 2025

Fixes #42

@ldemailly ldemailly changed the title Update README with duration flag information Update README with duration flag information and example Sep 11, 2025
@ldemailly ldemailly requested a review from Copilot September 11, 2025 21:23
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds documentation and example usage for duration flags in the CLI package. It demonstrates how to integrate the fortio.org/duration package with the existing CLI framework to support human-readable duration parsing.

  • Added import and usage of the duration package to show duration flag support
  • Updated example code to demonstrate duration flag creation and usage
  • Added documentation explaining the integration with the duration package

Reviewed Changes

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

File Description
sampleTool/main.go Added duration flag example and updated log output to show parsed duration
go.mod Added dependency on fortio.org/duration v1.0.1
README.md Updated documentation with duration flag information and example usage

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@ldemailly ldemailly merged commit 90051bb into main Sep 11, 2025
5 checks passed
@ldemailly ldemailly deleted the ldemailly-patch-2 branch September 11, 2025 21:26
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.

move fortio.org/tclock/duration here

1 participant