Skip to content

Commit 522ded5

Browse files
authored
chore: s.remove_staging_dirs() should only be called once (#362)
There is [an issue](https://github.com/googleapis/python-firestore/blob/master/owlbot.py#L60) in the `owlbot.py` file added in #352 in that [s.remove_staging_dirs()](https://github.com/googleapis/synthtool/blob/master/synthtool/transforms.py#L309) should only be called once after all the files are copied over. [get_staging_dirs()](https://github.com/googleapis/synthtool/blob/master/synthtool/transforms.py#L280) will only return staging directories that exist.
1 parent 6e2c899 commit 522ded5

File tree

1 file changed

+1
-5
lines changed

1 file changed

+1
-5
lines changed

owlbot.py

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -57,8 +57,6 @@ def update_fixup_scripts(library):
5757
update_fixup_scripts(library)
5858
s.move(library / "scripts")
5959

60-
s.remove_staging_dirs()
61-
6260
for library in s.get_staging_dirs(admin_default_version):
6361
if library.parent.absolute() == 'admin':
6462
s.move(
@@ -70,8 +68,6 @@ def update_fixup_scripts(library):
7068
update_fixup_scripts(library)
7169
s.move(library / "scripts")
7270

73-
s.remove_staging_dirs()
74-
7571
for library in s.get_staging_dirs(bundle_default_version):
7672
if library.parent.absolute() == 'bundle':
7773
s.replace(
@@ -251,4 +247,4 @@ def lint_setup_py(session):
251247
252248
*************
253249
Test Coverage"""
254-
)
250+
)

0 commit comments

Comments
 (0)