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.

Sign up for GitHub

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

Jump to bottom

Add funding option #345

Merged
merged 3 commits into from
Aug 21, 2022
Merged

Add funding option #345

merged 3 commits into from
Aug 21, 2022

Conversation

paescuj
Copy link
Collaborator

@paescuj paescuj commented Aug 10, 2022

No description provided.

@coveralls
Copy link

coveralls commented Aug 10, 2022 •
edited
Loading

Coverage Status

Coverage remained the same at 100.0% when pulling 70414d4 on paescuj:funding into 93e0891 on open-cli-tools:main.

@paescuj paescuj marked this pull request as ready for review August 10, 2022 13:37
@paescuj paescuj changed the title Add FUNDING.yml Add funding option Aug 11, 2022
gustavohenke
gustavohenke approved these changes Aug 20, 2022
View reviewed changes
Copy link
Member

@gustavohenke gustavohenke left a comment

Choose a reason for hiding this comment

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

thanks!

@paescuj
Copy link
Collaborator Author

paescuj commented Aug 21, 2022

Well, thank you 👈 Just make sure to set-up GitHub Sponsors on your profile!

@paescuj paescuj merged commit c6cbe6b into open-cli-tools:main Aug 21, 2022
@paescuj paescuj deleted the funding branch August 21, 2022 13:05
@paescuj
Copy link
Collaborator Author

paescuj commented Aug 21, 2022 •
edited
Loading

And apparently you then need to manually enable the sponsor button.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gustavohenke gustavohenke gustavohenke approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@paescuj @coveralls @gustavohenke