Skip to content

chore: Migrate logging synth.py to bazel #124

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.

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
merged 1 commit into from
Mar 30, 2020

Conversation

vam-google
Copy link
Contributor

@vam-google vam-google commented Mar 30, 2020

This PR migrates only synth.py but does not commit the regenerated files. The generation was tested and it works, the updated files are not commited due to breaking changes not related to bazel migration.

There is already an open PR with those braking changes: #111

This PR migrates only synth.py but does not commit the regenerated files. The generation was tested and it works, the updated files are not commited due to breaking changes not related to bazel migration. There is already an open PR with those braking changes: googleapis#111
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Mar 30, 2020
@vam-google
Copy link
Contributor Author

@chingor13 PTAL

@codecov
Copy link

codecov bot commented Mar 30, 2020

Codecov Report

Merging #124 into master will not change coverage by %.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #124   +/-   ##
=========================================
  Coverage     76.82%   76.82%           
  Complexity      530      530           
=========================================
  Files            42       42           
  Lines          3318     3318           
  Branches        212      212           
=========================================
  Hits           2549     2549           
  Misses          626      626           
  Partials        143      143           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e3c2b4c...c86b94d. Read the comment docs.

@chingor13 chingor13 merged commit c983911 into googleapis:master Mar 30, 2020
yoshi-automation added a commit that referenced this pull request Mar 30, 2020
c983911
commit c983911
Author: Vadym Matsishevskyi <25311427+vam-google@users.noreply.github.com>
Date:   Mon Mar 30 13:47:15 2020 -0700

    chore: Migrate logging synth.py to bazel (#124)

    This PR migrates only synth.py but does not commit the regenerated files. The generation was tested and it works, the updated files are not commited due to breaking changes not related to bazel migration. There is already an open PR with those braking changes: #111
yoshi-automation added a commit that referenced this pull request Apr 10, 2020
Source-Repo: googleapis/java-logging
Source-Sha: c983911
Source-Link: c983911
Author: Vadym Matsishevskyi <25311427+vam-google@users.noreply.github.com>
Date: Mon Mar 30 13:47:15 2020 -0700
Original-Commit-Message: chore: Migrate logging synth.py to bazel (#124)

This PR migrates only synth.py but does not commit the regenerated files. The generation was tested and it works, the updated files are not commited due to breaking changes not related to bazel migration. There is already an open PR with those braking changes: #111
yoshi-automation added a commit that referenced this pull request Apr 10, 2020
Source-Repo: googleapis/java-logging
Source-Sha: c983911
Source-Link: c983911
Author: Vadym Matsishevskyi <25311427+vam-google@users.noreply.github.com>
Date: Mon Mar 30 13:47:15 2020 -0700
Original-Commit-Message: chore: Migrate logging synth.py to bazel (#124)

This PR migrates only synth.py but does not commit the regenerated files. The generation was tested and it works, the updated files are not commited due to breaking changes not related to bazel migration. There is already an open PR with those braking changes: #111
yoshi-automation added a commit that referenced this pull request Apr 15, 2020
This PR migrates only synth.py but does not commit the regenerated files. The generation was tested and it works, the updated files are not commited due to breaking changes not related to bazel migration. There is already an open PR with those braking changes: #111

Source-Author: Vadym Matsishevskyi <25311427+vam-google@users.noreply.github.com>
Source-Date: Mon Mar 30 13:47:15 2020 -0700
Source-Repo: googleapis/java-logging
Source-Sha: c983911
Source-Link: c983911
yoshi-automation added a commit that referenced this pull request Apr 16, 2020
This PR migrates only synth.py but does not commit the regenerated files. The generation was tested and it works, the updated files are not commited due to breaking changes not related to bazel migration. There is already an open PR with those braking changes: #111

Source-Author: Vadym Matsishevskyi <25311427+vam-google@users.noreply.github.com>
Source-Date: Mon Mar 30 13:47:15 2020 -0700
Source-Repo: googleapis/java-logging
Source-Sha: c983911
Source-Link: c983911
yoshi-automation added a commit that referenced this pull request Apr 16, 2020
This PR migrates only synth.py but does not commit the regenerated files. The generation was tested and it works, the updated files are not commited due to breaking changes not related to bazel migration. There is already an open PR with those braking changes: #111

Source-Author: Vadym Matsishevskyi <25311427+vam-google@users.noreply.github.com>
Source-Date: Mon Mar 30 13:47:15 2020 -0700
Source-Repo: googleapis/java-logging
Source-Sha: c983911
Source-Link: c983911
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes This human has signed the Contributor License Agreement.
3 participants