Skip to content

Conversation

andreimerlescu
Copy link
Owner

@andreimerlescu andreimerlescu commented Jun 15, 2025

This PR activates the Usage function in the flag package by updating its signature and implementation across several files. Key changes include:

Changing the Usage method on figTree from returning a string to printing directly.
Updating the Core interface in types.go to match the new Usage signature.
Removing t.Log calls from tests to accommodate the new behavior of Usage, and updating the VERSION file.

@andreimerlescu andreimerlescu self-assigned this Jun 15, 2025
@Copilot Copilot AI review requested due to automatic review settings June 15, 2025 14:26
@andreimerlescu andreimerlescu added the enhancement New feature or request label Jun 15, 2025
Copy link
Contributor

@Copilot 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 activates the Usage function in the flag package by updating its signature and implementation across several files. Key changes include:

  • Changing the Usage method on figTree from returning a string to printing directly.
  • Updating the Core interface in types.go to match the new Usage signature.
  • Removing t.Log calls from tests to accommodate the new behavior of Usage, and updating the VERSION file.

Reviewed Changes

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

Show a summary per file
File Description
usage.go Changed Usage signature and implementation to print output directly
types.go Updated the Usage method signature in the Core interface
figtree.go Assigned the updated Usage function to the flag set
alias_test.go Removed t.Log calls for Usage reflecting the new behavior
VERSION Bumped version from v2.0.9 to v2.0.10
Comments suppressed due to low confidence (1)

alias_test.go:18

  • Since Usage now prints directly to standard output, consider capturing stdout in tests if verifying the help output is important.
t.Log(figs.Usage())

@andreimerlescu andreimerlescu merged commit ce21143 into main Jun 15, 2025
6 checks passed
@andreimerlescu andreimerlescu deleted the feature/usage branch June 15, 2025 14:29
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.

1 participant