Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

AFL License template is not identical to devdocs AFL License header #31

Open
matks opened this issue Oct 18, 2020 · 8 comments
Open

AFL License template is not identical to devdocs AFL License header #31

matks opened this issue Oct 18, 2020 · 8 comments

Comments

@matks
Copy link
Contributor

matks commented Oct 18, 2020

According to https://devdocs.prestashop.com/1.7/development/coding-standards/#module-files

AFL License header should have this line:
* @author PrestaShop SA and Contributors <contact@prestashop.com>

The AFL License template in this repository does not have this line.

@matks
Copy link
Contributor Author

matks commented Oct 18, 2020

@Quetzacoalt91 @ttoine Could you please confirm which version is right ? Devdocs or https://github.com/PrestaShopCorp/header-stamp/blob/master/assets/afl.txt ?

@Quetzacoalt91
Copy link
Contributor

I'll let Antoine answer the question, as I would have relied on him anyway.

@matks
Copy link
Contributor Author

matks commented Nov 2, 2020

@ttoine We need your expertise

@ttoine
Copy link

ttoine commented Nov 3, 2020

Well, it depends.

  • For everything in the PrestaShop project's GitHub, sure, it's * @author PrestaShop SA and Contributors <contact@prestashop.com>, because it's the open source project.
  • For PrestaShop Corp's GitHub, you must check first if all authors are PrestaShop SA employees or subcontractors (past and present).
    • If yes, the company is owner of all the copyright on the module, then you can keep * @author PrestaShop SA <contact@prestashop.com>.
    • If there is an external contribution on a module, then, you add and Contributors.
    • If you want to remove and contributors, then your team will have to rewrite the parts authored by independent contributors.

Is it clear now ?

@matks
Copy link
Contributor Author

matks commented Nov 3, 2020

@ttoine thank your for answer. So we actually have 2 usecases 😛 : AFL license header for PrestaShop-only modules and AFL license header for PrestaShop + contributors modules.

@ttoine
Copy link

ttoine commented Nov 3, 2020

Looks like that, yes.
Maybe my answer should go in some documentation for future reference?

@matks
Copy link
Contributor Author

matks commented Nov 5, 2020

@Quetzacoalt91 Would it be OK to have 2 AFL templates stored inside this repository, and let user choose the one he wants?

@Quetzacoalt91
Copy link
Contributor

Quetzacoalt91 commented Dec 9, 2020

We can have 2 different files yes.

I guess the difference on PrestaShop Corp would be if we have pulic or private repositories ? Because on a public repository, at any time we could have our first extenal contributor on the project.

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

No branches or pull requests

3 participants