Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added drivers command #221

Merged
merged 9 commits into from
Jun 4, 2024
Merged

Added drivers command #221

merged 9 commits into from
Jun 4, 2024

Conversation

jetroni
Copy link
Contributor

@jetroni jetroni commented May 15, 2024

added: drivers command (--list, --download)
added: drivers.yaml file
added: DriverHelper for drivers command
updated: README with newly added command

added: drivers.yaml file
added: DriverHelper for drivers command
updated: README with newly added command
@jetroni jetroni requested review from ailegion, nbesimi and Femi3211 May 15, 2024 10:15
@@ -413,6 +419,39 @@ private void diff(@CommandLine.Option(names = {"-s", "--source"}) String sourceN
}
}

@CommandLine.Command(name = "drivers", description = "Show available drivers for download", mixinStandardHelpOptions = true)
private void drivers(@CommandLine.Option(names = {"--list"}, description = "Used to list all available drivers") boolean isList,
@CommandLine.Option(names = {"--download"}, description = "Used to download selected driver by index") boolean isDownload,
Copy link
Contributor

Choose a reason for hiding this comment

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

Do you think we need a shortcut for --download ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, I agree, should we add -dl ?

@nbesimi nbesimi merged commit 12d2d72 into develop Jun 4, 2024
1 check passed
@nbesimi nbesimi deleted the feature/add_drivers_command branch June 5, 2024 09:13
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