Let's follow the same format used for --repo-url:

        self.container_build_parser.add_argument(
            '--repo-url',
            metavar="URL",
            help=('URL of yum repo file'),
            nargs='*')

default=None is redundant. Let's just remove it.

dest in this case is redundant, --signing-intent gets automatically converted to signing_intent.

:+1:

This trailing space should be kept here.

Same here.

This should not happen because of the group group = self.container_build_parser.add_mutually_exclusive_group() that prevents from using --compose-id and --signing_intent at same time.

I'm thinking if we can give same command line arguments error UX when giving both --compose-id and --repo-url.

I agree with all of cqi's comments. Let's remove the unnecessary explicit check for mutually exclusivity, as well as, add repo-url to the mutually exclusive group added.

Looks good.

Looks good to me as well. Thank you. :thumbsup:

Looks good to me.

Can someone merge this?

Commit c2853f4c fixes this pull-request