Skip to content

Conversation

@WyriHaximus
Copy link
Member

This will show the core team as authors under the dependencies on Packagist.

@WyriHaximus WyriHaximus added this to the v1.0.0 milestone Jul 5, 2020
@WyriHaximus WyriHaximus requested review from cboden, clue and jsor July 5, 2020 15:43
"homepage": "https://sorgalla.com/",
"email": "[email protected]"
},
{
Copy link
Member

Choose a reason for hiding this comment

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

Looking at my commit history for this repo, I've primary only done peer merges and the odd tooling cleaning. I would not consider myself an author of this repo.

Copy link
Member Author

Choose a reason for hiding this comment

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

@cboden Well the way I look at it, if you are on the team, reviewing code puts you at the same level of responsibility as writing code as by approving the code you say "I trust this code as if I wrote it myself and I take the responsibility for it as if I wrote it myself".

Copy link
Member

Choose a reason for hiding this comment

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

🤷‍♂️ ok 😄

@WyriHaximus WyriHaximus merged commit 7b4e687 into reactphp:master Jul 7, 2020
@WyriHaximus WyriHaximus deleted the add-reactphp-core-team-as-authors branch July 7, 2020 15:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants