Skip to content

Modules list args #5683

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

Merged
merged 1 commit into from
Dec 1, 2022
Merged

Conversation

jimingham
Copy link

Missing cherry-pick. When Dave added this patch, it was just to get the help right. But a subsequent change I made to require commands specify arguments - so that we could detect and give an error if arguments were provided to a command that doesn't take them - turned this into an error.

Register positional argument details in `CommandObjectTargetModulesList`.

I recently learned that `image list` takes a module name, but the help info
does not indicate this. With this change, `help image list` will show that it
accepts zero or more module names.

This makes it easier to get info about specific modules, without having to
find/grep through the full image list.

Reviewed By: DavidSpickett

Differential Revision: https://reviews.llvm.org/D125154

(cherry picked from commit 760298a)
@jimingham
Copy link
Author

@swift-ci please test

@jimingham jimingham merged commit f21cfd5 into swiftlang:stable/20220421 Dec 1, 2022
@jimingham jimingham deleted the modules-list-args branch December 1, 2022 03:16
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