Skip to content
This repository has been archived by the owner on Nov 3, 2020. It is now read-only.

Review author and contributors fields #398

Open
arkaitzgarro opened this issue Aug 27, 2019 · 1 comment
Open

Review author and contributors fields #398

arkaitzgarro opened this issue Aug 27, 2019 · 1 comment

Comments

@arkaitzgarro
Copy link
Contributor

As per npm documentation:

The “author” is one person. “contributors” is an array of people.

We are specifying "author": "WeTransfer folks",, which is not correct. We also may want to add ourselves as contributors, or add a generic dev account, like "contributors": ["WeTransfer folks <devs@wetransfer.com> (https://github.com/WeTransfer)"]

@seelts
Copy link

seelts commented Aug 28, 2019

Yep, as I've mentioned in the clipboard PR, I think we should not use author field for "WeTransfer folks".
Let's use contributors field for that.

As for author field and putting a real person's name there, I have nothing against it.
You can see other companies doing that.
For me it looks correct from the ethical point of view.
That for sure should be aligned with WeTransfer's policy regarding this.
And I will not be surprised if this is prohibited.

So, let's just decide if we want it and if we are allowed to do that.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants