-
-
Notifications
You must be signed in to change notification settings - Fork 3
Add template for running D2D on project build and source archive #30
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
Conversation
Signed-off-by: Keshav Priyadarshi <[email protected]>
|
@keshav-space We need some documentation on how to use this. |
@tdruez added usage example in the README |
Signed-off-by: Keshav Priyadarshi <[email protected]>
e9b600b to
30e80be
Compare
tdruez
left a comment
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.
It would be nice to add a workflow that displays the usage of the new template on a small Python or Java.
|
@keshav-space gentle ping... what's your ETA? |
Signed-off-by: Keshav Priyadarshi <[email protected]>
|
@tdruez We now have an end-to-end working example for Python in docs. |
|
@keshav-space That's good but instead of putting this in the docs, it should be in the workflows and running as part of the check to ensure everything works on code changes. |
Signed-off-by: Keshav Priyadarshi <[email protected]>
c2fc567 to
c0d4b5e
Compare
|
@tdruez We now have a running workflow that does source-binary mapping on https://github.com/bastikr/boolean.py using D2D template. |
|
@keshav-space Just to understand, why building the binaries in the context of this workflow instead of using already existing builds from the source project? |
|
@tdruez this would be used in cases where we don’t already have a release binary, but we want to run source binary mapping in a pull/merge request or manual trigger. |
Resolves: aboutcode-org/scancode.io#1774