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

Replace gomock import with go.uber.org/mock #302

Merged
merged 1 commit into from
Jul 27, 2023
Merged

Conversation

afritzler
Copy link
Member

Proposed Changes

  • Update the MOCKGEN_VERSION from v1.6.0 to v0.2.0 in the Makefile
  • Update the github.com/golang/mock dependency version in go.mod to v0.2.0
  • Update the github.com/golang/mock/gomock import to go.uber.org/mock/gomock in pkg/driver/node_test.go
  • Update the github.com/golang/mock/gomock import to go.uber.org/mock/gomock in pkg/utils/mount/mock_mountutils_unix.go
  • Update the github.com/golang/mock/gomock import to go.uber.org/mock/gomock in pkg/utils/os/mock_osutils_unix.go

@afritzler afritzler requested a review from a team as a code owner July 27, 2023 13:48
@github-actions github-actions bot added size/S enhancement New feature or request labels Jul 27, 2023
…mock dependency to go.uber.org/mock

- Update the `MOCKGEN_VERSION` from `v1.6.0` to `v0.2.0` in the `Makefile`
- Update the `github.com/golang/mock` dependency version in `go.mod` to `v0.2.0`
- Update the `github.com/golang/mock/gomock` import to `go.uber.org/mock/gomock` in `pkg/driver/node_test.go`
- Update the `github.com/golang/mock/gomock` import to `go.uber.org/mock/gomock` in `pkg/utils/mount/mock_mountutils_unix.go`
- Update the `github.com/golang/mock/gomock` import to `go.uber.org/mock/gomock` in `pkg/utils/os/mock_osutils_unix.go`

Signed-off-by: Andreas Fritzler <[email protected]>
@github-actions github-actions bot added size/M and removed size/S labels Jul 27, 2023
@afritzler afritzler merged commit 37986ee into main Jul 27, 2023
7 checks passed
@afritzler afritzler deleted the enh/switch-gomock branch July 27, 2023 13:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request size/M
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant