Skip to content

New CLI command: search#2576

Draft
shaahji wants to merge 1 commit into
mainfrom
shaahji/clisearch
Draft

New CLI command: search#2576
shaahji wants to merge 1 commit into
mainfrom
shaahji/clisearch

Conversation

@shaahji

@shaahji shaahji commented Jul 17, 2026

Copy link
Copy Markdown
Collaborator

New CLI command: search

  • A specialized implementation of "run" CLI command that can built the config from command line args and run it.
  • Also, updated base.py to support command line args with both underscores and hyphens (aliasing only with full backward compatibility).

Checklist before requesting a review

  • Add unit tests for this change.
  • Make sure all tests can pass.
  • Update documents if necessary.
  • Lint and apply fixes to your code by running lintrunner -a
  • Is this a user-facing change? If yes, give a description of this change to be included in the release notes.

(Optional) Issue link

@github-advanced-security github-advanced-security 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.

lintrunner found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.

* A specialized implementation of "run" CLI command that can built the
  config from command line args and run it.
* Also, updated base.py to support command line args with both
  underscores and hyphens (aliasing only with full backward
  compatibility).
@shaahji
shaahji force-pushed the shaahji/clisearch branch from 0b1dc72 to 252a190 Compare July 17, 2026 20:12
Comment thread test/cli/test_search.py
stop_when_goals_met=False,
include_pass_params=None,
)
assert WorkflowSearchCommand._build_search_strategy_overrides(args) == {}
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.

2 participants