From 1cf170751a5c2500b4b02dae79669e1ae6f9efe2 Mon Sep 17 00:00:00 2001 From: Jarek Potiuk Date: Wed, 11 May 2022 10:14:08 +0200 Subject: [PATCH] Fix assuming "Feature" answer on CI when generating docs We have now different answers posisble when generating docs, and for testing we assume we answered randomly during the generation of documentation. --- .../prepare_provider_packages.py | 16 ++++++++++++++-- 1 file changed, 14 insertions(+), 2 deletions(-) diff --git a/dev/provider_packages/prepare_provider_packages.py b/dev/provider_packages/prepare_provider_packages.py index 44ee3a9f9065c..0e3e27cd9aa2b 100755 --- a/dev/provider_packages/prepare_provider_packages.py +++ b/dev/provider_packages/prepare_provider_packages.py @@ -37,6 +37,7 @@ from functools import lru_cache from os.path import dirname, relpath from pathlib import Path +from random import choice from shutil import copyfile from typing import Any, Dict, Iterable, List, NamedTuple, Optional, Set, Tuple, Union @@ -1138,12 +1139,23 @@ class TypeOfChange(Enum): SKIP = "s" -def get_type_of_changes() -> TypeOfChange: +def get_type_of_changes(answer: Optional[str]) -> TypeOfChange: """ Ask user to specify type of changes (case-insensitive). :return: Type of change. """ given_answer = "" + if answer and answer.lower() in ["yes", "y"]: + # Simulate all possible non-terminal answers + return choice( + [ + TypeOfChange.DOCUMENTATION, + TypeOfChange.BUGFIX, + TypeOfChange.FEATURE, + TypeOfChange.BREAKING_CHANGE, + TypeOfChange.SKIP, + ] + ) while given_answer not in [*[t.value for t in TypeOfChange], "q"]: console.print( "[yellow]Type of change (d)ocumentation, (b)ugfix, (f)eature, (x)breaking " @@ -1223,7 +1235,7 @@ def update_release_notes( console.print() return False else: - type_of_change = get_type_of_changes() + type_of_change = get_type_of_changes(answer=answer) if type_of_change == TypeOfChange.DOCUMENTATION: if isinstance(latest_change, Change): mark_latest_changes_as_documentation_only(provider_package_id, latest_change)