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

Various smaller adjustments #42

Closed
wants to merge 20 commits into from

Conversation

centic9
Copy link
Member

@centic9 centic9 commented Oct 30, 2018

Use the latest version of JUnit
Some unused imports removed
Set compiler to Java 1.7 to have a defined version of Java for the resulting binaries
Ignore one more file created by IntelliJ

centic9 and others added 20 commits October 30, 2018 07:14
…c are actually instrumented

This uses new dependency on Mockito for testing with mock objects.
…does

This triggers a problem because we record open in Files.newDirectoryStream, but close in a different place, so a replaced FileSystemProvider actually uses a different object then.
This should probably be reverted anyway...
@basil
Copy link
Member

basil commented Mar 9, 2022

Hey @centic9, thank you for all your contributions! I've merged whatever looked finished from this branch. If you have any additional contributions, please open separate PRs for them. Thanks!

@centic9
Copy link
Member Author

centic9 commented Mar 9, 2022

Thanks a lot for dissecting and merging! I will take a look at my local repository if there are more changes that are ready for submitting.

@centic9 centic9 closed this Mar 9, 2022
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.

4 participants