Skip to content

Commit 67cf899

Browse files
committed
address review feedback
1 parent 67d6450 commit 67cf899

1 file changed

Lines changed: 4 additions & 3 deletions

File tree

scripts/configure_release_please/configure_release_please.py

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -76,14 +76,15 @@ def configure_release_please_manifest(
7676
with open(release_please_manifest, "r") as f:
7777
manifest_json = json.load(f)
7878
for package_dir in package_dirs:
79-
if not package_supports_gapic_version(package_dir):
80-
continue
8179
if f"packages/{package_dir.name}" not in manifest_json:
8280
manifest_json[f"packages/{package_dir.name}"] = "0.0.0"
8381

8482
gapic_version_file = next(package_dir.rglob("**/gapic_version.py"), None)
8583
if gapic_version_file is None:
86-
raise Exception("Failed to find gapic_version.py")
84+
if package_supports_gapic_version(package_dir):
85+
raise Exception("Failed to find gapic_version.py")
86+
else:
87+
continue
8788
version = get_version_for_package(gapic_version_file)
8889
# check the version in gapic_version.py and update if newer than the default which is
8990
# 0.0.0 or 0.1.0.

0 commit comments

Comments
 (0)