potiuk commented on code in PR #31830: URL: https://github.com/apache/airflow/pull/31830#discussion_r1401250414
########## dev/breeze/src/airflow_breeze/commands/release_management_commands.py: ########## @@ -16,22 +16,27 @@ # under the License. from __future__ import annotations +import logging import os import re import shlex import shutil import sys import textwrap import time +from collections import defaultdict from copy import deepcopy from datetime import datetime from pathlib import Path from subprocess import DEVNULL from typing import IO, Any, Generator, NamedTuple import click +import semver +from github import Github, UnknownObjectException from rich.progress import Progress from rich.syntax import Syntax +from tests.test_utils.providers import get_provider_min_airflow_version Review Comment: BTW. Breeze has quite comprehensive and "unitized" set of utils - each group of utils has it's own module in utils. I am from the school where those util classes should be split to many independend small set of closely related utils that can be imported and used independently from each other (avoid all kinds of circular dependencies and diamond relationships. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
