Skip to content

refactor: do manual args check#19

Merged
ninoseki merged 1 commit intomainfrom
do-manual-args-check
Jun 16, 2025
Merged

refactor: do manual args check#19
ninoseki merged 1 commit intomainfrom
do-manual-args-check

Conversation

@ninoseki
Copy link
Collaborator

@ninoseki ninoseki commented Jun 14, 2025 •
edited
Loading

Do manual args check to show usage on the args error (#17).

Before

$ urlscan search
Error: accepts 1 arg(s), received 0

After

$ urlscan search
Usage:
  urlscan search <query> [flags]

Examples:
  urlscan search <query>
  echo "<query>" | urlscan search -

...

@ninoseki ninoseki requested review from cdnsyseng and fw42 June 14, 2025 04:10
@ninoseki ninoseki merged commit 59fea13 into main Jun 16, 2025
4 checks passed
@ninoseki ninoseki deleted the do-manual-args-check branch June 16, 2025 09:23
@ninoseki ninoseki mentioned this pull request Jul 4, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@fw42 fw42 fw42 approved these changes

@cdnsyseng cdnsyseng cdnsyseng approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ninoseki @fw42 @cdnsyseng