Skip to content
This repository has been archived by the owner on Feb 28, 2024. It is now read-only.

chore: remove obsolete workarounds in owlbot.py #564

Merged
merged 6 commits into from
Oct 17, 2023
Merged

Conversation

parthea
Copy link
Contributor

@parthea parthea commented Apr 14, 2023

The changes in this PR were validated by running these commands:

  1. In a clone of googleapis/googleapis, run the following bazel build commands
bazel build //google/cloud/asset/v1:asset-v1-py
bazel build //google/cloud/asset/v1p5beta1:asset-v1p5beta1-py
bazel build //google/cloud/asset/v1p2beta1:asset-v1p2beta1-py
bazel build //google/cloud/asset/v1p1beta1:asset-v1p1beta1-py
  1. Check out the branch in this PR and run the following commands to copy the generated code and run post processing
docker run --rm --user $(id -u):$(id -g)   -v $(pwd):/repo   -v $HOME/git/googleapis/bazel-bin:/bazel-bin   gcr.io/cloud-devrel-public-resources/owlbot-cli:latest copy-bazel-bin   --source-dir /bazel-bin --dest /repo
docker run --user $(id -u):$(id -g) --rm -v $(pwd):/repo -w /repo [gcr.io/cloud-devrel-public-resources/owlbot-python:latest](http://gcr.io/cloud-devrel-public-resources/owlbot-python:latest)

@parthea parthea requested a review from a team as a code owner April 14, 2023 12:52
@product-auto-label product-auto-label bot added size: s Pull request size is small. api: cloudasset Issues related to the googleapis/python-asset API. labels Apr 14, 2023
@product-auto-label product-auto-label bot added size: m Pull request size is medium. and removed size: s Pull request size is small. labels Oct 16, 2023
@parthea
Copy link
Contributor Author

parthea commented Oct 16, 2023

@parthea parthea added the owlbot:run Add this label to trigger the Owlbot post processor. label Oct 17, 2023
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Oct 17, 2023
@parthea parthea merged commit 23f71e8 into main Oct 17, 2023
24 checks passed
@parthea parthea deleted the parthea-patch-2 branch October 17, 2023 20:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
api: cloudasset Issues related to the googleapis/python-asset API. size: m Pull request size is medium.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants