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

chore(package.json): remove author, maintainers, etc. #279

Merged
merged 2 commits into from
Aug 19, 2021

Conversation

brodybits
Copy link
Member

@brodybits brodybits commented Aug 19, 2021

since they are visible on github/npm level and don't need to additionally be synced/maintained inside package.json, as discussed for followup in PR #278.

Copy link
Member

@karfau karfau left a comment

Choose a reason for hiding this comment

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

@brodybits I wonder if it would make sense to also drop the "author" property.
At least it feels strange that it points to somebody that is no longer involved/we have not been able to contact and it being the only direct contact information left in this file.

@brodybits
Copy link
Member Author

Agreed on my part to remove the authors as well, thanks.

@brodybits brodybits changed the title chore(package.json): remove maintainers & contributors chore(package.json): remove authors, maintainers, etc. Aug 19, 2021
@brodybits brodybits changed the title chore(package.json): remove authors, maintainers, etc. chore(package.json): remove author, maintainers, etc. Aug 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants