Skip to content

DOC Add additional donation options to About page #31379

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

Merged
merged 2 commits into from
Jun 10, 2025

Conversation

ShauryaDusht
Copy link
Contributor

@ShauryaDusht ShauryaDusht commented May 18, 2025

Reference Issues/PRs

References #30826

What does this implement/fix? Explain your changes.

Updates the "Donating to the project" section on the About page to include additional donation options:

  • Added GitHub Sponsors information
  • Included corporate giving programs (Benevity and Open Collective)
  • Maintained existing NumFOCUS donation information
  • Formatted as bullet points for readability

Any other comments?

Open to feedback on wording or formatting if needed.

Copy link

github-actions bot commented May 18, 2025

✔️ Linting Passed

All linting checks passed. Your pull request is in excellent shape! ☀️

Generated for commit: 6a60552. Link to the linter CI: here

@ShauryaDusht
Copy link
Contributor Author

Hi @reshamas
Please review my PR

@betatim
Copy link
Member

betatim commented Jun 3, 2025

Thanks for picking up this issue and working on it. Looks reasonable to me

@ShauryaDusht
Copy link
Contributor Author

Updated as suggested — removed Benevity and Open Collective corporate donation references.

@reshamas
Copy link
Member

reshamas commented Jun 6, 2025

@reshamas
Copy link
Member

reshamas commented Jun 6, 2025

@betatim This PR is ready to be merged.
However, for the above description, can we change it from "Fixes xxxxx" to "References xxxxx"?
(I do not have edit rights to do that.)

I would like to keep the issue open - there are still some changes I would like to make which require internal knowledge of a core contributor. I was working on those, but I have not been able to build the docs locally.

@betatim
Copy link
Member

betatim commented Jun 10, 2025

Thanks for making the PR @ShauryaDusht and for reviewing it @reshamas. Merging

@betatim betatim merged commit 8bec6d0 into scikit-learn:main Jun 10, 2025
37 checks passed
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.

3 participants