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
Add docker file #178
Add docker file #178
Conversation
Signed-off-by: Pratik Dey <pratikrocks.dey11@gmail.com>
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.
Can you write a bit of documentation as to how to go about using deltacode with Docker? So simple examples would be nice
Okay then i am adding in the RTD related Pr |
docker-compose.yml
Outdated
@@ -0,0 +1,28 @@ | |||
# SPDX-License-Identifier: Apache-2.0 | |||
# | |||
# http://nexb.com and https://github.com/nexB/scancode.io |
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.
I reckon you copied this from ScanCode.io... but there is no need to carry the scancode.io references
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.
This also should be changed in Dockerfile not just here in docker-compose.yml. There are other references to scancode.io too in the comments of both files that should be adapted.
It would be nice to have those changes soon and merged, as I would like to update my local installation and avoid creating my own Dockerfile locally.
Thanks a lot.
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.
@pombredanne , @ddmesh I changed those licenses Thanks :)
Signed-off-by: Pratik Dey <pratikrocks.dey11@gmail.com>
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.
LGTM... merging at last!
Added
DockerFile
docker-compose.yml
Signed-off-by: Pratik Dey pratikrocks.dey11@gmail.com