Skip to content

chore(librarian): update configure_state_yaml.py to avoid duplicate entries #14452

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
parthea merged 1 commit into from
Sep 17, 2025
Merged

chore(librarian): update configure_state_yaml.py to avoid duplicate entries #14452

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 7 additions & 0 deletions scripts/configure_state_yaml/configure_state_yaml.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,14 @@ def configure_state_yaml() -> None:
with open(LIBRARIAN_YAML, "r") as state_yaml_file:
state_dict = yaml.safe_load(state_yaml_file)

existing_library_ids = {library["id"] for library in state_dict.get("libraries", [])}

for package_name in packages_to_onboard["packages_to_onboard"]:
# Check for duplication
if package_name in existing_library_ids:
print(f"Skipping package '{package_name}' as it already exists in state.yaml.")
continue

package_path = Path(PACKAGES_DIR / package_name).resolve()
api_paths = []
for individual_metadata_file in package_path.rglob(f"**/{GAPIC_METADATA_JSON}"):
Expand Down
Loading