dash-licenses icon indicating copy to clipboard operation
dash-licenses copied to clipboard

Report more information in the output

Open waynebeaton opened this issue 5 years ago • 1 comments

I'm envisioning an update to the process for handling third party libraries where the committer need only create a single CQ that lists all of the third party content that needs review.

Currently the tool outputs a list of "problematic" third party content as ClearlyDefined URLs.

In the process of identifying those URLs, we actually discard a lot of information. I'd like to turn the implementation around a bit and actually gather the information that we find together and provide it in some form in the output. Right now the "support" classes make decisions on what to do. I'm thinking that we have the "support" classes just gather the information and then make decisions based on the totality of what is gathered. Then, again, the output would summarize the totality of what we discover (e.g., "such-and-such was rejected in CQ 1234, but ClearlyDefined gives it a passing grade" along with URLs, etc.)

I'm thinking that we could then send that output to the IP Team and have them sort it out. The simplest form would be to just copy and paste the output into a CQ. After we have that, we can think about more extensive changes, potentially involving IPZilla itself.

waynebeaton avatar Oct 01 '20 17:10 waynebeaton

I've made a change in the way that information is gathered by the model and have tweaked the current output a bit.

I've added a "review" option that generates markdown output that's appropriate as a comment in GitLab. My intention is to experiment with using GitLab as a means of tracking the review of third party content (and perhaps as an eventual replacement for IPZilla). I've created one example and will create additional examples as I continue to experiment with this format.

At some point, I'll extend the experiment to automate (or semi-automate) the creation of an issue using GitLab APIs.

waynebeaton avatar Nov 09 '20 00:11 waynebeaton