Skip to content

Conversation

@hejazizo
Copy link
Collaborator

No description provided.

@hejazizo hejazizo requested a review from arf1372 December 30, 2021 01:50
@hejazizo hejazizo self-assigned this Dec 30, 2021
@CLAassistant
Copy link

CLAassistant commented Dec 30, 2021

CLA assistant check
All committers have signed the CLA.

Copy link

@arf1372 arf1372 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see you added a CLA checker bot. Would you insert appropriate instructions on this to CONTRIBUTING.md?

I want to mention this license (CC-BY-NC-SA 4) prevents ANY commercial reuse of this material for ANYONE EXCEPT OWNER. If you mind allow some "fair commercial usage policy", please add it to README file.
You also would prefer a "dual-licensing scheme" for these situations.

Also, ensure materials used in this repo allow application of a NC (or even a CC) license at all! Attributions and citations must be clear and right. Can you - at least - refer to an attribution list to check compatibility and to avoid copyright infringements/plagiarism?

P.S: I do these precise considerations because: 1st. CC licenses are Irrevocable. 2nd. Once this branch merges and the work publishes, the license applies to this and all foregoing works as long as these license terms reside here.

@arf1372
Copy link

arf1372 commented Dec 30, 2021

Regarding to attribution/citation part here:

Also, ensure materials used in this repo allow application of a NC (or even a CC) license at all! Attributions and citations must be clear and right. Can you - at least - refer to an attribution list to check compatibility and to avoid copyright infringements/plagiarism?

This means every single material come from any outside source must have clear and correct citation and bibliography on each file.

@arf1372
Copy link

arf1372 commented Jan 30, 2022

@hejazizo As this repo now has an Expat license (MIT), would you mind closing this PR?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants