NoHarm icon indicating copy to clipboard operation
NoHarm copied to clipboard

Add licence summary

Open realpixelcode opened this issue 3 years ago • 9 comments
trafficstars

Adds an easy-to-understand summary of the licence. Fixes #72.

realpixelcode avatar Jul 26 '22 13:07 realpixelcode

Just read it. Very concise and digestible. Now I bet the core team is going to probably ask on how to integrate it into the main license for ease of access, or probably include a reference in the README.md.

ghost avatar Jul 27 '22 12:07 ghost

Just read it. Very concise and digestible. Now I bet the core team is going to probably ask on how to integrate it into the main license for ease of access, or probably include a reference in the README.md.

Just added a link to the README.md :)

realpixelcode avatar Jul 27 '22 13:07 realpixelcode

@realpixelcode, I was just rereading your license summary, and I was thinking that if this PR gets merged before mine (#79), I think I could possibly make a template for the summary as well (I would have to ask the core team because it would require different API). What do you think?

ghost avatar Jul 28 '22 14:07 ghost

In principle, I find that a good idea, I'm just wondering how you would do that? A script would most likely not be sufficiently "intelligent" enough to generate a summary out of a new licence text itself.

realpixelcode avatar Jul 28 '22 15:07 realpixelcode

I mean to modify the existing context to build the summary from its own template. This is much more simpler than machine learning.

ghost avatar Jul 28 '22 20:07 ghost

Ok, go ahead then, I guess 😊

realpixelcode avatar Jul 28 '22 20:07 realpixelcode

Sorry for the late response. I thought I responded earlier.

ghost avatar Jul 28 '22 20:07 ghost

(edited for grammar)

Still waiting for review from the core team. They may not want to "template-ify" the summary.

ghost avatar Jul 28 '22 20:07 ghost

Sorry for the late response. I thought I responded earlier.

No worries, mate :) Aren't we multiple time zones apart anyway? 😄 Also, others take much longer. 🤷

realpixelcode avatar Jul 28 '22 20:07 realpixelcode

Hello @realpixelcode. I can see that this PR is not getting merged; the same is happening for me, so I decided to prefix the title my PR (#79 ) with [Stale!], and I'm thinking about closing it entirely and keeping the branch for when someone wants to re-open it.

ghost avatar Sep 16 '22 15:09 ghost

Sorry for the delay here, thanks for the contribution!

No worries :)

realpixelcode avatar Sep 17 '22 13:09 realpixelcode