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

Activating Open Collective #235

Merged
merged 4 commits into from Feb 9, 2019
Merged

Conversation

monkeywithacupcake
Copy link
Contributor

Hi, I'm making updates for Open Collective. Either you or a supporter signed this repo up for Open Collective. This pull request adds backers and sponsors from your Open Collective https://opencollective.com/badgen-service ❤️

It adds two badges at the top to show the latest number of backers and sponsors. It also adds placeholders so that the avatar/logo of new backers/sponsors can automatically be shown without having to update your README.md. [more info]. See how it looks on this repo.
We have also added a postinstall script to let people know after npm|yarn install that you are welcoming donations (optional). [More info]
You can also add a "Donate" button to your website and automatically show your backers and sponsors there with our widgets. Have a look here: https://opencollective.com/widgets

P.S: As with any pull request, feel free to comment or suggest changes. The only thing "required" are the placeholders on the README because we believe it's important to acknowledge the people in your community that are contributing (financially or with code!).

Thank you for your great contribution to the open source community. You are awesome! 🙌
And welcome to the open collective community! 😊

Come chat with us in the #opensource channel on https://slack.opencollective.com - great place to ask questions and share best practices with other open source sustainers!

@vercel
Copy link

vercel bot commented Feb 3, 2019

This pull request is automatically deployed with Now.
To access deployments, click Details below or on the icon next to each push.

@wopian
Copy link
Contributor

wopian commented Feb 3, 2019

I'd recommend removing the postinstall script - it increases the production dependency from 5.7mb to 17.3mb o_o

https://packagephobia.now.sh/result?p=opencollective

Copy link
Member

@amio amio left a comment

Choose a reason for hiding this comment

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

Awesome, thanks @monkeywithacupcake !

I agree with @wopian on removing postinstall script, then we are good to go 👍

remove Open Collective postinstall
@monkeywithacupcake
Copy link
Contributor Author

Removed :)

@amio amio merged commit c371141 into badgen:master Feb 9, 2019
@vercel vercel bot temporarily deployed to staging February 9, 2019 13:56 Inactive
@amio
Copy link
Member

amio commented Feb 9, 2019

Thanks! @monkeywithacupcake

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.

None yet

3 participants