Skip to content

Switched to go.uber.org/mock in v2#29

Merged
aureagdm merged 5 commits into
masterfrom
renovate/all
Nov 13, 2024
Merged

Switched to go.uber.org/mock in v2#29
aureagdm merged 5 commits into
masterfrom
renovate/all

Conversation

@aureagdm

Copy link
Copy Markdown
Contributor

Switched to go.uber.org/mock in v2 due to resolve snyk issues found in component-httpstats introduced by github.com/asecurityteam/httpstats@v2.4.0

https://app.snyk.io/org/atlassian-security-open-source/project/3a0fa179-11a9-4f22-aa18-67fa4b08e188/pr-check/15bbbbc8-cf28-4b99-963a-ed6571975f3f

All 8 issues come from crypto which gets removed by this upgrade. Additionally golang/mock is archived and the suggested path is to switch to the fork.

@aureagdm
aureagdm requested a review from a team as a code owner November 13, 2024 19:44
@codecov

codecov Bot commented Nov 13, 2024

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.89%. Comparing base (8daaf52) to head (b7318ba).

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #29      +/-   ##
==========================================
- Coverage   90.69%   89.89%   -0.80%     
==========================================
  Files           4        4              
  Lines         430      495      +65     
==========================================
+ Hits          390      445      +55     
- Misses         32       42      +10     
  Partials        8        8              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@aureagdm
aureagdm removed the request for review from a team November 13, 2024 20:25
@aureagdm
aureagdm merged commit bd1aa92 into master Nov 13, 2024
@aureagdm
aureagdm deleted the renovate/all branch November 13, 2024 20:37
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