search

dokan-dev / dokan-java

Dokan Java Wrapper
GNU Lesser General Public License v3.0
51 stars 28 forks source link

Reduced dependency on Guava, SLF4J and Apache Commons. Upgrades. Grad… #30

Closed hrstoyanov closed 4 years ago

hrstoyanov commented 4 years ago

…le 6.1.1, JUnit 5.6.0, JNA 5.5.0, SLF4J 1.7.30. Minor gradle.build and code cleanup.

Incorporated feedback from @infeo from previous similar pull request (old one was killed)

This is still pointing to master branch

infeo commented 4 years ago

Regarding the branch: I asked to change the default one, but up to now noone answered. So this will go directly the master branch.

hrstoyanov commented 4 years ago

In future, you can add back an IntelliJ editor style/foramt file .editorconfig, see IntelliJ docs

Requested changes implemented.

hrstoyanov commented 4 years ago

I incorporated your feedback - take a look again.

Separately, in my MASTER branch here, I will be doing some more aggressive changes, which will not be backward compatible, if ouy are interested...

hrstoyanov commented 4 years ago

@infeo were you bale to review the latest changes?

infeo commented 4 years ago

Ah, thanks for the reminder. Got nearly through last week, but then forgot about the merge.

Thanks for hinting to your branch, i'll watch it from time to time!