Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Remove guava #87

Closed
jglaszka opened this issue Jul 3, 2023 · 0 comments · Fixed by #93
Closed

Remove guava #87

jglaszka opened this issue Jul 3, 2023 · 0 comments · Fixed by #93

Comments

@jglaszka
Copy link
Collaborator

jglaszka commented Jul 3, 2023

Tradukisto does not use guava cache - only a few helper methods from com.google.common.*.

todo:

  • Research if it is possible to replace it with plain java (with not reinventing the wheel)
  • Remove guava from dependencies
pawelkowalski92 added a commit to pawelkowalski92/tradukisto that referenced this issue Jul 4, 2023
@pawelkowalski92
allegro#87 Remove guava dependency, add JUnit platform in gradle to e…
64c1ee0 
…nable test execution
pawelkowalski92 added a commit to pawelkowalski92/tradukisto that referenced this issue Jul 4, 2023
@pawelkowalski92
allegro#87 Replace guava dependencies
8bcd3cf
@pawelkowalski92 pawelkowalski92 mentioned this issue Jul 4, 2023
Merged
jglaszka added a commit that referenced this issue Jul 4, 2023
@jglaszka
Merge pull request #93 from pawelkowalski92/feature/87-remove-guava
1402e29 
#87 Remove guava
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

#87 Remove guava pawelkowalski92/tradukisto
1 participant
@jglaszka