-
-
Notifications
You must be signed in to change notification settings - Fork 3
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
Feature Request: Migrate Java IO to Java NIO APIs where applicable #34
Comments
@mattmook Hi Matt! If you're interested in a PR for this, just let me know. |
davidburstromspotify
added a commit
to davidburstromspotify/markdown-lint
that referenced
this issue
Dec 14, 2020
davidburstromspotify
added a commit
to davidburstromspotify/markdown-lint
that referenced
this issue
Dec 15, 2020
mattmook
pushed a commit
that referenced
this issue
Dec 16, 2020
@mattmook You can close this FR now :) |
Will publish this next week and close then. Planning to split out #32 at the same time |
Thanks! I guess Christmas got rightfully in the way :) |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
TL;DR
Use
java.nio.file.Path
andjava.nio.file.Files
operations instead ofjava.io.File
wherever possible.Benefits
My particular reason for this suggestion is that the
ValidRelativeLinksRule
is trying to resolve files outside of the analysed file. If (for particular reasons) only the analysed file is available on the real file system, the rule will raise a warning. In my case, the referenced file is available in a virtual file system, which is not readable through Java IO.Cannot be migrated
The APIs provided by Gradle are currently centered around using
File
s. That is OK, sinceMarkdownLintPlugin
simply has to callprojectDir.toPath
to convert into the Java NIO realm, given that everything else is migrated.References:
The text was updated successfully, but these errors were encountered: