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

build: Lock to mockery minor version #1038

Merged
merged 1 commit into from Jun 15, 2022

Conversation

Eric-Warehime
Copy link
Contributor

Summary

Mockery bumped the golang version without doing a major version bump of their package. This breaks us since we're just pulling v2 latest. More discussion vektra/mockery#456 about this decision and why they say not to use mockery via go install/go generate.

Test Plan

CI

@Eric-Warehime Eric-Warehime changed the title Lock to mockery minor version build: Lock to mockery minor version Jun 15, 2022
@codecov
Copy link

codecov bot commented Jun 15, 2022

Codecov Report

Merging #1038 (5bca3f4) into develop (c9f5dc0) will decrease coverage by 0.05%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop    #1038      +/-   ##
===========================================
- Coverage    59.58%   59.52%   -0.06%     
===========================================
  Files           45       45              
  Lines         8353     8353              
===========================================
- Hits          4977     4972       -5     
- Misses        2918     2922       +4     
- Partials       458      459       +1     
Impacted Files Coverage Δ
fetcher/fetcher.go 33.77% <0.00%> (-2.20%) ⬇️

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 c9f5dc0...5bca3f4. Read the comment docs.

@Eric-Warehime Eric-Warehime merged commit 475f416 into algorand:develop Jun 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants