Skip to content

tools: add mockery to tools.go and remove mockery version strings#6787

Merged
mergify[bot] merged 1 commit intomasterfrom
wb/mockery-tools
Jul 30, 2021
Merged

tools: add mockery to tools.go and remove mockery version strings#6787
mergify[bot] merged 1 commit intomasterfrom
wb/mockery-tools

Conversation

@williambanfield
Copy link
Contributor

This change aims to keep versions of mockery consistent across developer laptops.

This change adds mockery to the tools.go file so that its version can be managed consistently in the go.mod file.

Additionally, this change temporarily disables adding mockery's version number to generated files. There is an outstanding issue against the mockery project related to the version string behavior when running from go get. I have created a pull request to fix this issue in the mockery project.
see: vektra/mockery#397

@williambanfield williambanfield added the S:automerge Automatically merge PR when requirements pass label Jul 30, 2021
@codecov
Copy link

codecov bot commented Jul 30, 2021

Codecov Report

Merging #6787 (ef12f0b) into master (02f8e4c) will decrease coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #6787      +/-   ##
==========================================
- Coverage   62.62%   62.58%   -0.04%     
==========================================
  Files         306      306              
  Lines       40385    40385              
==========================================
- Hits        25290    25274      -16     
- Misses      13321    13339      +18     
+ Partials     1774     1772       -2     
Impacted Files Coverage Δ
abci/client/client.go 36.17% <ø> (ø)
internal/consensus/reactor.go 70.86% <ø> (+0.54%) ⬆️
internal/p2p/peer.go 55.86% <ø> (+1.67%) ⬆️
internal/p2p/transport.go 100.00% <ø> (ø)
internal/statesync/stateprovider.go 0.00% <ø> (ø)
proxy/app_conn.go 26.66% <ø> (ø)
proxy/client.go 28.12% <ø> (ø)
state/services.go 33.33% <ø> (ø)
state/store.go 56.55% <ø> (ø)
internal/statesync/dispatcher.go 87.93% <0.00%> (-5.53%) ⬇️
... and 16 more

@mergify mergify bot merged commit 4e96c6b into master Jul 30, 2021
@mergify mergify bot deleted the wb/mockery-tools branch July 30, 2021 20:47
@williambanfield williambanfield restored the wb/mockery-tools branch September 9, 2022 16:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S:automerge Automatically merge PR when requirements pass

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants