Skip to content

Conversation

@rhatdan
Copy link
Member

@rhatdan rhatdan commented Nov 30, 2016

Current code only handled commands not the options.

@rhatdan rhatdan mentioned this pull request Nov 30, 2016

_skopeo_skopeo() {
local options_with_args="
--username
Copy link
Member

Choose a reason for hiding this comment

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

these won't be here after #254

Copy link
Member Author

Choose a reason for hiding this comment

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

Ok, I will remove them once I have this merged.

Copy link
Member

Choose a reason for hiding this comment

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

Ok, I will remove them once I have this merged.

Do you want this PR merged first? cause I already merged #254 😄

@rhatdan
Copy link
Member Author

rhatdan commented Nov 30, 2016

A little script I wrote to check for options:

 for i in $(./skopeo --generate-bash-completion); do echo "========= $i ============"; ./skopeo $i --help; done

@rhatdan
Copy link
Member Author

rhatdan commented Nov 30, 2016

Updated to match #254

@runcom
Copy link
Member

runcom commented Nov 30, 2016

LGTM

@runcom
Copy link
Member

runcom commented Nov 30, 2016

@rhatdan could you rebase this one and I'll merge?

Current code only handled commands not the options.
@rhatdan
Copy link
Member Author

rhatdan commented Nov 30, 2016

Rebased.

@runcom runcom merged commit 6b616d1 into containers:master Nov 30, 2016
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 8, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants