diff --git a/metadata-ingestion/src/datahub/cli/docker_cli.py b/metadata-ingestion/src/datahub/cli/docker_cli.py index cb4bc8bda5..74d4505ca0 100644 --- a/metadata-ingestion/src/datahub/cli/docker_cli.py +++ b/metadata-ingestion/src/datahub/cli/docker_cli.py @@ -11,7 +11,7 @@ import tempfile import time from enum import Enum from pathlib import Path -from typing import Dict, List, NoReturn, Optional +from typing import Dict, List, Optional import click import click_spinner @@ -87,18 +87,6 @@ def docker() -> None: pass -def _print_issue_list_and_exit( - issues: List[str], header: str, footer: Optional[str] = None -) -> NoReturn: - click.secho(header, fg="bright_red") - for issue in issues: - click.echo(f"- {issue}") - if footer: - click.echo() - click.echo(footer) - sys.exit(1) - - @docker.command() @upgrade.check_upgrade @telemetry.with_telemetry() diff --git a/metadata-ingestion/src/datahub/cli/quickstart_versioning.py b/metadata-ingestion/src/datahub/cli/quickstart_versioning.py index 0ba3c025bb..dc7bd1ecb4 100644 --- a/metadata-ingestion/src/datahub/cli/quickstart_versioning.py +++ b/metadata-ingestion/src/datahub/cli/quickstart_versioning.py @@ -29,7 +29,7 @@ def _is_it_a_version(version: str) -> bool: :param version: The string to check. :return: True if the string is a valid version, False otherwise. """ - return re.match(r"v?\d+\.\d+(\.\d+)?", version) is not None + return re.match(r"^v?\d+\.\d+(\.\d+)?$", version) is not None class QuickstartVersionMappingConfig(BaseModel):