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.

Sign up for GitHub

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

Jump to bottom

scancode-toolkit2/scancode-toolkit/samples shows in the html app for all scans #152

Closed
pombredanne opened this issue Dec 10, 2015 · 4 comments
Closed

scancode-toolkit2/scancode-toolkit/samples shows in the html app for all scans #152

pombredanne opened this issue Dec 10, 2015 · 4 comments
Assignees
@jdaguil
Labels
fixed pending review GUI and outputs

Comments

@pombredanne
Copy link
Member

This is a bug and there should be a test.
@pombredanne
Copy link
Member Author

This was introduced by cb6a669 here: cb6a669#diff-e9488ab53b7aea277034313f9a6611cfL77

@pombredanne
Copy link
Member Author

This is affecting master

pombredanne added a commit that referenced this issue Dec 10, 2015
@pombredanne
Fix for #152
c1790b8
@pombredanne
Copy link
Member Author

Fixed and released in https://github.com/nexB/scancode-toolkit/releases/tag/v1.4.3

@pombredanne
Copy link
Member Author

We still need tests. @jdaguil I am passing this over to you

jdaguil added a commit that referenced this issue Dec 29, 2015
@jdaguil
#152: added test case for html-app scanned output path to ensure corr…
da82dcd 
…ect path is in results
@jdaguil jdaguil closed this as completed Jan 22, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jdaguil jdaguil

Labels
fixed pending review GUI and outputs
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@pombredanne @jdaguil