- Notifications
You must be signed in to change notification settings - Fork44
Use Path in the Scanner API#226
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
base:master
Are you sure you want to change the base?
Uh oh!
There was an error while loading.Please reload this page.
Conversation
@olamy could this be a first step? |
Also look atapache/maven-shared-utils#54. IMHO we should only implement in one library.... |
The most notable thing I see is that it is two years old now and no progress at all... |
I'm a bit skeptical about breaking the API and replacing the return type of the |
This would be a minimal change to move the API of
Scannerto thePathAPIThis could be some kind of minimal change that could be merged as base for larger changes like: