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.

Sign up for GitHub

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

Jump to bottom

Do not allow multiple input apps #128

Merged
merged 1 commit into from
Dec 5, 2023
Merged

Do not allow multiple input apps #128

merged 1 commit into from
Dec 5, 2023

Conversation

eemcmullan
Copy link
Collaborator

No description provided.

@eemcmullan
do not allow multiple input apps
b4c908e 
Signed-off-by: Emily McMullan <[email protected]>
shawn-hurley
shawn-hurley approved these changes Dec 4, 2023
View reviewed changes
Copy link
Contributor

@shawn-hurley shawn-hurley left a comment

Choose a reason for hiding this comment

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

I thought for sure there as a some option to add to the flag in pflag to do this for use, but there is not! thanks this looks good to me

@eemcmullan
Copy link
Collaborator Author

I thought for sure there as a some option to add to the flag in pflag to do this for use, but there is not! thanks this looks good to me

@shawn-hurley I spent way too much time trying to find something like that

@eemcmullan eemcmullan merged commit e35c2bd into konveyor:main Dec 5, 2023
2 checks passed
@eemcmullan eemcmullan deleted the source-input branch January 23, 2024 16:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@shawn-hurley shawn-hurley shawn-hurley approved these changes

@pranavgaikwad pranavgaikwad Awaiting requested review from pranavgaikwad

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@eemcmullan @shawn-hurley