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

[README] Remove Bitcoin and Bountysource donation options #10491

Merged
merged 2 commits into from
Oct 27, 2023
Merged

[README] Remove Bitcoin and Bountysource donation options #10491

merged 2 commits into from
Oct 27, 2023

Conversation

TobiGr
Copy link
Member

@TobiGr TobiGr commented Oct 13, 2023

What is it?

  • Bugfix (user facing)
  • Feature (user facing)
  • Codebase improvement (dev facing)
  • Meta improvement to the project (dev facing)

Description of the changes in your PR

Remove Bitcoin and Bountysource donation options for now.

  • Bitcoin due to its high energy consumption
  • Bountysource because their platform has issues

Due diligence

@TobiGr TobiGr added the meta Related to the project but not strictly to code label Oct 13, 2023
@github-actions github-actions bot added the size/medium PRs with less than 250 changed lines label Oct 13, 2023
@TobiGr TobiGr changed the title Readme [README] Remove Bitcoin and Bountysource donation options Oct 13, 2023
opusforlife2
opusforlife2 approved these changes Oct 26, 2023
View reviewed changes
Copy link
Collaborator

@opusforlife2 opusforlife2 left a comment

Choose a reason for hiding this comment

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

All changes except one look fine to me. If that's a non-issue, this can be merged as is.

doc/README.asm.md Show resolved Hide resolved
@Stypox Stypox merged commit 6d69451 into dev Oct 27, 2023
3 checks passed
@TobiGr TobiGr deleted the readme branch October 27, 2023 07:56
@Stypox Stypox mentioned this pull request Nov 16, 2023
Closed
21 tasks
@AudricV AudricV mentioned this pull request Feb 7, 2024
Closed
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@opusforlife2 opusforlife2 opusforlife2 approved these changes

Assignees
No one assigned
Labels
meta Related to the project but not strictly to code size/medium PRs with less than 250 changed lines
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@TobiGr @opusforlife2 @Stypox