Skip to content

Client defaults to device linsting#332

Merged
jenstopp merged 1 commit into
mainfrom
feat/dutctl-default-list-and-auto-resolve
Jul 1, 2026
Merged

Client defaults to device linsting#332
jenstopp merged 1 commit into
mainfrom
feat/dutctl-default-list-and-auto-resolve

Conversation

@RiSKeD

@RiSKeD RiSKeD commented May 13, 2026

Copy link
Copy Markdown
Contributor

resolves #293

@RiSKeD RiSKeD changed the title Feat/dutctl default list and auto resolve Feat: default list and auto resolve May 13, 2026
@jenstopp jenstopp changed the title Feat: default list and auto resolve Client defaults to device linsting Jun 11, 2026

@jenstopp jenstopp left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Pls, reduce it to the default-to-list part. See #293 (comment)

@RiSKeD RiSKeD force-pushed the feat/dutctl-default-list-and-auto-resolve branch 2 times, most recently from df15d90 to c8bc202 Compare June 30, 2026 10:25
jenstopp
jenstopp previously approved these changes Jun 30, 2026

@jenstopp jenstopp left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Please just change the synopsis slightly.

Comment thread cmds/dutctl/dutctl.go Outdated
@RiSKeD RiSKeD force-pushed the feat/dutctl-default-list-and-auto-resolve branch from e5828dc to 795925b Compare July 1, 2026 08:39
@RiSKeD

RiSKeD commented Jul 1, 2026

Copy link
Copy Markdown
Contributor Author

@jenstopp fixed 👍

jenstopp
jenstopp previously approved these changes Jul 1, 2026
@RiSKeD RiSKeD force-pushed the feat/dutctl-default-list-and-auto-resolve branch from 795925b to 6ad5e2c Compare July 1, 2026 09:58
Extract the dispatch logic from start() into a small testable helper,
cover it with baseline dispatch tests, and make a bare `dutctl` (no
positional arguments) default to the list command. The usage text
documents the new no-args form.

Signed-off-by: Fabian Wienand <fabian.wienand@blindspot.software>
@RiSKeD RiSKeD force-pushed the feat/dutctl-default-list-and-auto-resolve branch from 6ad5e2c to 004ff73 Compare July 1, 2026 10:07
@RiSKeD RiSKeD requested a review from jenstopp July 1, 2026 10:08
@jenstopp jenstopp merged commit 352a464 into main Jul 1, 2026
11 checks passed
@jenstopp jenstopp deleted the feat/dutctl-default-list-and-auto-resolve branch July 1, 2026 12:52
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.

Default behaviour with device selection

2 participants