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

do not pin deps to exact versions #2844

Merged
merged 1 commit into from Nov 15, 2018
Merged

do not pin deps to exact versions #2844

merged 1 commit into from Nov 15, 2018

Conversation

melekes
Copy link
Contributor

@melekes melekes commented Nov 15, 2018

(at least those with releases)

because

  • they are locked in .lock file already
  • individual dependencies can be updated with dep ensure -update XXX
  • review process (and ^^^) should help us prevent accidental updates

Closes #2798

  • Updated all relevant documentation in docs
  • Updated all code comments where relevant
  • Wrote tests
  • Updated CHANGELOG_PENDING.md

because
- they are locked in .lock file already
- individual dependencies can be updated with `dep ensure -update XXX`
- review process (and ^^^) should help us prevent accidental updates

Closes #2798
@codecov-io
Copy link

Codecov Report

Merging #2844 into develop will increase coverage by 0.18%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop    #2844      +/-   ##
===========================================
+ Coverage    62.23%   62.41%   +0.18%     
===========================================
  Files          212      212              
  Lines        17255    17224      -31     
===========================================
+ Hits         10738    10750      +12     
+ Misses        5614     5572      -42     
+ Partials       903      902       -1
Impacted Files Coverage Δ
privval/tcp_server.go 78.57% <0%> (-2.86%) ⬇️
libs/db/remotedb/grpcdb/server.go 0% <0%> (ø) ⬆️
consensus/reactor.go 67.89% <0%> (+0.23%) ⬆️
consensus/state.go 80.02% <0%> (+0.23%) ⬆️
consensus/wal_generator.go 79.43% <0%> (+0.27%) ⬆️
p2p/pex/pex_reactor.go 72.99% <0%> (+0.32%) ⬆️

Copy link
Contributor

@ValarDragon ValarDragon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is great! I think this is a good idea, since why would we enforce the exact revision of these libs for any tm dependency.


[[constraint]]
name = "google.golang.org/grpc"
version = "=1.13.0"
version = "~1.13.0"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this being considered a serialization library? We should move it to the other section then.

@ebuchman ebuchman merged commit e93b492 into develop Nov 15, 2018
@ebuchman ebuchman deleted the anton/deps branch November 15, 2018 19:40
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.

None yet

4 participants