Skip to content

deps: bump viper to 1.4.0 and logfmt to 0.4.0#3950

Merged
melekes merged 3 commits intomasterfrom
marko/updateDeps
Sep 5, 2019
Merged

deps: bump viper to 1.4.0 and logfmt to 0.4.0#3950
melekes merged 3 commits intomasterfrom
marko/updateDeps

Conversation

@tac0turtle
Copy link
Contributor

@tac0turtle tac0turtle commented Sep 4, 2019

Changesets:

Signedoff-by: Marko Baricevic marbar3778@yahoo.com

  • Referenced an issue explaining the need for the change
  • Updated all relevant documentation in docs
  • Updated all code comments where relevant
  • Wrote tests
  • Updated CHANGELOG_PENDING.md

- depndabot couldnt update a few deps as they needed to beupdated together.
- PRs: #3946, 3938 can be closed with this PR

Signedoff-by: Marko Baricevic <marbar3778@yahoo.com>
logger.Info("foo", "baz baz", "bar")
msg := strings.TrimSpace(buf.String())
if !strings.Contains(msg, logfmt.ErrInvalidKey.Error()) {
if !strings.Contains(msg, "foo") {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

github.com/spf13/jwalterweatherman v1.0.0 // indirect
github.com/spf13/pflag v1.0.3 // indirect
github.com/spf13/viper v1.0.0
github.com/spf13/viper v1.4.0
Copy link
Contributor Author

Choose a reason for hiding this comment

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

all these changes came from upgrading this version of viper

@tac0turtle tac0turtle added the T:dependencies Pull requests that update a dependency file label Sep 4, 2019
@melekes melekes changed the title Depanbot upgrade issues deps: bump viper to 1.4.0 and logfmt to 0.4.0 Sep 5, 2019
@tac0turtle tac0turtle self-assigned this Sep 5, 2019
@codecov-io
Copy link

Codecov Report

Merging #3950 into master will decrease coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #3950      +/-   ##
==========================================
- Coverage      67%   66.95%   -0.05%     
==========================================
  Files         219      219              
  Lines       18473    18458      -15     
==========================================
- Hits        12377    12359      -18     
+ Misses       5179     5177       -2     
- Partials      917      922       +5
Impacted Files Coverage Δ
libs/log/tm_logger.go 35.48% <0%> (-9.68%) ⬇️
libs/events/events.go 93.2% <0%> (-4.86%) ⬇️
behaviour/reporter.go 44.44% <0%> (-4.05%) ⬇️
libs/log/tmfmt_logger.go 85.18% <0%> (-3.71%) ⬇️
privval/signer_endpoint.go 81.33% <0%> (-2.67%) ⬇️
blockchain/v0/reactor.go 77.83% <0%> (-1.89%) ⬇️
p2p/pex/pex_reactor.go 83.76% <0%> (-0.58%) ⬇️
blockchain/v0/pool.go 80.65% <0%> (-0.33%) ⬇️
consensus/reactor.go 77.96% <0%> (ø) ⬆️
privval/signer_server.go 95.65% <0%> (ø) ⬆️
... and 2 more

Copy link
Contributor

@melekes melekes left a comment

Choose a reason for hiding this comment

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

verified

github.com/aead/siphash v1.0.1/go.mod h1:Nywa3cDsYNNK3gaciGTWPwHt0wlpNV15vwmswBAUSII=
github.com/alecthomas/template v0.0.0-20160405071501-a0175ee3bccc/go.mod h1:LOuyumcjzFXgccqObfd/Ljyb9UuFJ6TxHnclSeseNhc=
github.com/alecthomas/units v0.0.0-20151022065526-2efee857e7cf/go.mod h1:ybxpYRFXyAe+OPACYpWeL0wqObRcbAqCMya13uyzqw0=
github.com/armon/consul-api v0.0.0-20180202201655-eb2c6b5be1b6/go.mod h1:grANhF5doyWs3UAsr3K4I6qtAmlQcZDesFNEHPZAzj8=
Copy link
Contributor

Choose a reason for hiding this comment

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

🆘

github.com/cespare/xxhash v1.1.0/go.mod h1:XrSqR1VqqWfGrhpAt58auRo0WTKS1nRRg3ghfAqPWnc=
github.com/client9/misspell v0.3.4/go.mod h1:qj6jICC3Q7zFZvVWo7KLAzC3yx5G7kyvSDkc90ppPyw=
github.com/coreos/bbolt v1.3.2/go.mod h1:iRUV2dpdMOn7Bo10OQBFzIJO9kkE559Wcmn+qkEiiKk=
github.com/coreos/etcd v3.3.10+incompatible/go.mod h1:uF7uidLiAD3TWHmW31ZFd/JWoc32PjwdhPthX9715RE=
Copy link
Contributor

Choose a reason for hiding this comment

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

🏝

@melekes melekes merged commit 4ba49a4 into master Sep 5, 2019
@melekes melekes deleted the marko/updateDeps branch September 5, 2019 07:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T:dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants