Skip to content

Bump gogo protobuf deps to v1.3.3#553

Closed
shahankhatch wants to merge 1 commit into
mainfrom
shahan/Remove_gogo_protobuf_dependency
Closed

Bump gogo protobuf deps to v1.3.3#553
shahankhatch wants to merge 1 commit into
mainfrom
shahan/Remove_gogo_protobuf_dependency

Conversation

@shahankhatch

Copy link
Copy Markdown
Contributor

Closes: #XXX

Description


For contributor use:

  • Targeted PR against correct branch (see CONTRIBUTING.md)
  • Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
  • Code follows the module structure standards.
  • Wrote unit and integration tests
  • Updated relevant documentation (docs/) or specification (x/<module>/spec/)
  • Added relevant godoc comments.
  • Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
  • Re-reviewed Files changed in the Github PR explorer

For admin use:

  • Added appropriate labels to PR (ex. WIP, R4R, docs, etc)
  • Reviewers assigned
  • Squashed all commits, uses message "Merge pull request #XYZ: [title]" (coding standards)

@codecov

codecov Bot commented Jan 19, 2021

Copy link
Copy Markdown

Codecov Report

Merging #553 (65821b0) into main (2a71985) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #553   +/-   ##
=======================================
  Coverage   11.58%   11.58%           
=======================================
  Files           8        8           
  Lines         673      673           
=======================================
  Hits           78       78           
  Misses        586      586           
  Partials        9        9           

@shahankhatch

Copy link
Copy Markdown
Contributor Author

Closing in favor #555

@shahankhatch shahankhatch deleted the shahan/Remove_gogo_protobuf_dependency branch January 19, 2021 21:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants