Skip to content

Conversation

@simone-stacks
Copy link

@simone-stacks simone-stacks commented Nov 10, 2025

Description

This PR aims to add usage messages to the clarity-cli commands that previously didn't have one.

generate_address, repl and eval_raw now require exactly no arguments to be passed as parameter, or they will fire the "Usage: " command.

Applicable issues

Checklist

  • Test coverage for new or modified code paths
  • Changelog is updated
  • Required documentation changes (e.g., docs/rpc/openapi.yaml and rpc-endpoints.md for v2 endpoints, event-dispatcher.md for new events)
  • New clarity functions have corresponding PR in clarity-benchmarking repo

@CLAassistant
Copy link

CLAassistant commented Nov 10, 2025

CLA assistant check
All committers have signed the CLA.

@simone-stacks simone-stacks marked this pull request as ready for review November 11, 2025 11:05
@simone-stacks simone-stacks requested review from a team as code owners November 11, 2025 11:05
jcnelson
jcnelson previously approved these changes Nov 11, 2025
Copy link
Contributor

@brice-stacks brice-stacks left a comment

Choose a reason for hiding this comment

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

This looks good, I just had a minor formatting comment.

brice-stacks
brice-stacks previously approved these changes Nov 12, 2025
Copy link
Contributor

@brice-stacks brice-stacks left a comment

Choose a reason for hiding this comment

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

lgtm!

@simone-stacks simone-stacks dismissed stale reviews from brice-stacks and jcnelson via f5868e7 November 12, 2025 14:46
Copy link
Contributor

@brice-stacks brice-stacks left a comment

Choose a reason for hiding this comment

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

lgtm!

Copy link

@francesco-stacks francesco-stacks left a comment

Choose a reason for hiding this comment

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

LGTM!

@jcnelson jcnelson added this pull request to the merge queue Nov 14, 2025
Merged via the queue into stacks-network:develop with commit 2662a10 Nov 14, 2025
3 of 4 checks passed
@github-actions
Copy link

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 22, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants