Skip to content

Commit 6524f37

Browse files
authored
chore(librarian): remove samples/generated_samples/snippet_metadata_from preserve_regex (#14469)
This commit fixes a regression where snippet_metadata files, such as `packages/google-cloud-video-live-stream/samples/generated_samples/snippet_metadata_google.cloud.video.livestream.v1.json`, were not being updated during generation (e.g., in PR #14465). The root cause was the inclusion of `samples/generated_samples/snippet_metadata_` in the preserve_regex list. While this regex was initially used to reduce diffs during the transition from OwlBot to Librarian, it inadvertently prevented necessary updates to the generated snippet metadata.
1 parent 7926fdd commit 6524f37

File tree

3 files changed

+0
-14
lines changed

3 files changed

+0
-14
lines changed

.generator/cli.py

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -224,11 +224,6 @@ def _clean_up_files_after_post_processing(output: str, library_id: str):
224224
): # pragma: NO COVER
225225
os.remove(gapic_version_file)
226226

227-
for snippet_metadata_file in glob.glob(
228-
f"{output}/{path_to_library}/samples/generated_samples/snippet_metadata*.json"
229-
): # pragma: NO COVER
230-
os.remove(snippet_metadata_file)
231-
232227

233228
def handle_generate(
234229
librarian: str = LIBRARIAN_DIR,

.librarian/state.yaml

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,6 @@ libraries:
1515
- samples/README.txt
1616
- tar.gz
1717
- gapic_version.py
18-
- samples/generated_samples/snippet_metadata_
1918
- scripts/client-post-processing
2019
- samples/snippets/README.rst
2120
- tests/system
@@ -36,7 +35,6 @@ libraries:
3635
- samples/README.txt
3736
- tar.gz
3837
- gapic_version.py
39-
- samples/generated_samples/snippet_metadata_
4038
- scripts/client-post-processing
4139
- samples/snippets/README.rst
4240
- tests/system
@@ -57,7 +55,6 @@ libraries:
5755
- samples/README.txt
5856
- tar.gz
5957
- gapic_version.py
60-
- samples/generated_samples/snippet_metadata_
6158
- scripts/client-post-processing
6259
- samples/snippets/README.rst
6360
- tests/system
@@ -78,7 +75,6 @@ libraries:
7875
- samples/README.txt
7976
- tar.gz
8077
- gapic_version.py
81-
- samples/generated_samples/snippet_metadata_
8278
- scripts/client-post-processing
8379
- samples/snippets/README.rst
8480
- tests/system
@@ -103,7 +99,6 @@ libraries:
10399
- samples/README.txt
104100
- tar.gz
105101
- gapic_version.py
106-
- samples/generated_samples/snippet_metadata_
107102
- scripts/client-post-processing
108103
- samples/snippets/README.rst
109104
- tests/system
@@ -125,7 +120,6 @@ libraries:
125120
- samples/README.txt
126121
- tar.gz
127122
- gapic_version.py
128-
- samples/generated_samples/snippet_metadata_
129123
- scripts/client-post-processing
130124
- samples/snippets/README.rst
131125
- tests/system
@@ -147,7 +141,6 @@ libraries:
147141
- samples/README.txt
148142
- tar.gz
149143
- gapic_version.py
150-
- samples/generated_samples/snippet_metadata_
151144
- scripts/client-post-processing
152145
- samples/snippets/README.rst
153146
- tests/system
@@ -168,7 +161,6 @@ libraries:
168161
- samples/README.txt
169162
- tar.gz
170163
- gapic_version.py
171-
- samples/generated_samples/snippet_metadata_
172164
- scripts/client-post-processing
173165
- samples/snippets/README.rst
174166
- tests/system

scripts/configure_state_yaml/configure_state_yaml.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -87,7 +87,6 @@ def configure_state_yaml() -> None:
8787
"samples/README.txt",
8888
"tar.gz",
8989
"gapic_version.py",
90-
"samples/generated_samples/snippet_metadata_",
9190
"scripts/client-post-processing",
9291
"samples/snippets/README.rst",
9392
"tests/system",

0 commit comments

Comments
 (0)