Added has_license and has_copyright to delta object

[shubhscoder]

Fixes #109
Signed-off-by: Shubham shubhamsangamnerkar9@gmail.com

[shubhscoder]

@MaJuRG , this is my first Pull request, please suggest me any required changes. Thank you !

[steven-esser]

@shubhscoder This is the correct first start. However, we want to populate these new field values properly in the right places.

See the various license and copyright functions in https://github.com/nexB/deltacode/blob/develop/src/deltacode/utils.py

Where we run delta.update(blah blah) we will want update these new flags for our Delta object and remove the "factors" that are added at each of these conditions.

[shubhscoder]

@MaJuRG , I have tried updating has_license and has copyright. The remaining delta.update would I guess go with addition of status right ? Also I know why the tests are failing because we have removed the scoring system. Should I try updating and addition of "status" in this PR itself? Then probably I can try updating the tells together. Also I am not sure why DCO is failing. The expected and received signs are exactly same.
Thank you so much !

[pombredanne]

Nit: please use an imperative style and not the past tense for your commit title. Also suffix it with the ticket number. And then explain in the body what this commit does
e.g. not Added has_license and has_copyright to delta object
but rather Add has_license/has_copyright to delta object #109

See https://github.com/nexB/aboutcode/wiki/Writing-good-commit-messages

[pombredanne]

also your git user config should match your signoff for the DCO bot to be happy ;)

[shubhscoder]

Nit: please use an imperative style and not the past tense for your commit title. Also suffix it with the ticket number. And then explain in the body what this commit does
e.g. not Added has_license and has_copyright to delta object
but rather Add has_license/has_copyright to delta object #109

See https://github.com/nexB/aboutcode/wiki/Writing-good-commit-messages

Thank you so much! I updated the commit message.

[shubhscoder]

also your git user config should match your signoff for the DCO bot to be happy ;)

@pombredanne , the "expected" and the "got" fields are exactly same. Also I checked with my local git, the global configuration is same as the signoff.

Thank you!

[pombredanne]

@shubhscoder the DCO bot may have a bug ... but in any case you need to sign with your full name. Thank you for fixing this

[shubhscoder]

Okay I ll do that. Thank you!

…

On Fri, Mar 22, 2019, 9:57 PM Philippe Ombredanne ***@***.***> wrote: @shubhscoder <https://github.com/shubhscoder> the DCO bot may have a bug ... but in any case you need to sign with your full name. Thank you for fixing this — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#114 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ARofxSs65EO_a_kprmQpanTJ2Q6m0BYqks5vZQSHgaJpZM4b3qq1> .

[steven-esser]

Closing due to stagnation.