-
Notifications
You must be signed in to change notification settings - Fork 62
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
[colic] Filter dir paths and softlinks from analysis #59
Conversation
95beb79
to
f1bbd6b
Compare
This code filters directory paths from the colic analysis. This change is needed to prevent failures in scancode-cli. Backend version is now set to 0.5.1 Signed-off-by: Valerio Cosentino <valcos@bitergia.com>
f1bbd6b
to
a8f1d02
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@valeriocos Thanks for the fix.
I've tested this locally and it works fine!
Thank you for reviewing it @inishchith . I launched an execution yesterday and found another error. I'm checking it.
|
The problem is probably related to the fact that The command below shows that the file is correctly recognized as a "file"
The command below shows that the
|
This code filters softlinks from the colic analysis. This change is needed to prevent failures in scancode-cli (issue chaoss#55) Signed-off-by: Valerio Cosentino <valcos@bitergia.com>
Tested on the full history of ELK, and no error popped up. |
This PR filters directory paths and softlinks from the colic analysis. This change is needed to prevent failures in scancode-cli (issue #55).
Backend version is now set to 0.5.2