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

Ensure we don't hash directory when using globs #1328

Merged
merged 1 commit into from
Jun 25, 2020
Merged

Conversation

Duhemm
Copy link
Collaborator

@Duhemm Duhemm commented Jun 24, 2020

Globs allow us to define the maximum depth to search for source files.
When we reach this maximum depth, the Files.walkFileTree will call
visitFile on directories that cannot be traversed.

Previously, we were adding these directories to the list of items to
hash, which would cause an exception to be thrown.

Globs allow us to define the maximum depth to search for source files.
When we reach this maximum depth, the `Files.walkFileTree` will call
`visitFile` on directories that cannot be traversed.

Previously, we were adding these directories to the list of items to
hash, which would cause an exception to be thrown.
@Duhemm Duhemm added bug A defect or misbehaviour. task / compile labels Jun 24, 2020
@Duhemm Duhemm requested a review from olafurpg June 24, 2020 12:58
Copy link
Contributor

@olafurpg olafurpg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice catch! LGTM 👍

@Duhemm Duhemm merged commit 34110ad into master Jun 25, 2020
@Duhemm Duhemm deleted the source-glob-depth branch June 25, 2020 08:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug A defect or misbehaviour. task / compile
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants