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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update test dependencies to newer versions #250

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ILikeYourHat
Copy link

Hello 馃憢 I found your plugin very interesting and I want to use it in my company's project. But before this would happen, I must be sure that it is still well maintained and there would be no issues during future collaboration, like submitting bugs or accepting changes from community. So I've prepared a small contribution, I would be very happy if you accept it and prove that this plugin is still getting some love by it's creators 馃槈

Changes:

  • bump com.google.truth:truth to 1.4.2 (IntelliJ reported that the previous version had some security vulnerabilities)
    • afaik no additional changes were necessary because of it
  • replace com.nhaarman:mockito-kotlin with org.mockito.kotlin:mockito-kotlin
    • Nhaarman's library was integrated into official Mockito organization, see link
    • the newest version of this library is 5.X. However, this major version requires Java 11, and your project is still on Java 8. Because of this, I've updated mockito-kotlin only to the 4.X version, which is still compatible with Java 8.

@CLAassistant
Copy link

CLAassistant commented Apr 26, 2024

CLA assistant check
All committers have signed the CLA.

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.

None yet

2 participants