Skip to content
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

Changed the protobuf version to 3.6.1 to accomodate latest bazel version #111

Closed
wants to merge 2 commits into from

Conversation

chamorajg
Copy link
Contributor

@chamorajg chamorajg commented Jun 3, 2019

Protobuf in the WORKSPACE has been changed and the bazel test cases are currently running fine locally. Closes #111.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
@googlebot googlebot added the cla: yes Contributor signed Google CLA label Jun 3, 2019
@codecov-io
Copy link

codecov-io commented Jun 3, 2019

Codecov Report

Merging #111 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           master    #111   +/-   ##
======================================
  Coverage    90.6%   90.6%           
======================================
  Files          31      31           
  Lines        2715    2715           
  Branches      432     432           
======================================
  Hits         2460    2460           
  Misses        177     177           
  Partials       78      78

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 574d3dc...0df5fc6. Read the comment docs.

@cweill
Copy link
Contributor

cweill commented Jun 3, 2019

Looks good to me, I'll merge it in.

cweill pushed a commit that referenced this pull request Jun 4, 2019
…ion | Closes PR #111.

Protobuf in the WORKSPACE has been changed and the bazel test cases are currently running fine locally.
Copybara import of the project:

  - 0df5fc6 Changed the protobuf version to 3.6.1 to accomodate lates... by chandramouli <chanduiyer.raja@gmail.com>
  - 64985e0 Merge branch 'master' into master by Charles Weill <weill@google.com>
  - 5fed4cd Merge 64985e0 into 6eb71... by chandramoulirajagopalan <chanduiyer.raja@gmail.com>

PiperOrigin-RevId: 251455788
@cweill
Copy link
Contributor

cweill commented Jun 4, 2019

Thanks for this. It's now merged in master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes Contributor signed Google CLA
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants