Conversation
|
Hi @chrisdecker1201, thanks for your contribution! This looks like a useful feature. Can you please add the signoff to your commits so that this can be merged? (see here Also, I have one question out of curiosity: I would assume that you use the tools-python as a library and as you also spoke up for the feature to merge SPDX documents: How or in what context are you using the tools? |
935f24c to
415c3b0
Compare
Signed-off-by: Christian Decker <christian.decker@homag.com>
Signed-off-by: Christian Decker <christian.decker@homag.com>
415c3b0 to
71607b1
Compare
|
@meretp Your assumption is correct 😄. We are getting a SPDX file from a third-party application and have to add additional SPDX information to this file. Therefore our developers should checkin this additional SPDX information into the repository and we are developing a tool for combining the files in the CI/CD process. Additionally we use this to validate the SPDX file. |
meretp
left a comment
There was a problem hiding this comment.
Thanks for adding the sign-off! I have some small remarks on the last changes.
Signed-off-by: Christian Decker <christian.decker@homag.com>
meretp
left a comment
There was a problem hiding this comment.
Thanks for addressing the remarks, this looks good to me! And also thanks for sharing your usecase!
No description provided.