human
I wrote the below code

    from .util import glob_files_locally
    source_files = [c for c in glob_files_locally(source, pattern)]
    if dryrun:
        logger = get_az_logger(__name__)
        logger.warning('upload    files    to    file    share')
        logger.warning('                account    %s', client.account_name)
        logger.warning('                        share    %s', destination)
        logger.warning('                        total    %d', len((source_files or [])))
        logger.warning('    operations')
        for f in (source_files or []):
            logger.warning('        -    %s    =>    %s', *f)
        return []
    def _upload_action(source_pair):
        dir_name = os.path.dirname(source_pair[1])
        file_name = os.path.basename(source_pair[1])
        _make_directory_in_files_share(client, destination, dir_name)
        create_file_args = {'share_name': destination, 'directory_name': dir_name, 'file_name': file_name, 'local_file_path': source_pair[0], 'content_settings': content_settings, 'metadata': metadata, 'max_connections': max_connections}
        if supported_api_version(ResourceType.DATA_STORAGE, min_api='2016-05-31'):
            create_file_args['validate_content'] = validate_content
        client.create_file_from_path(**create_file_args)
        return client.make_file_url(destination, dir_name, file_name)
    return list((_upload_action(f) for f in source_files))

What's a good function header?

=====
Here's a good function header?
assistant
#Here's a good function header.

def storage_file_upload_batch(client, destination, source, pattern=None, dryrun=False, validate_content=False, content_settings=None, max_connections=1, metadata=None):