Skip to content

Conversation

admtnnr
Copy link
Contributor

@admtnnr admtnnr commented Nov 1, 2014

Allow authors to submit PRs to include their projects in the blacklist.yml to prevent them from accepting tips. By using a YAML file checked into Git we can propagate the blacklist to forks as well.

  • Prevent new projects from being created from blacklist.
  • Prevent tips from being given to existing projects that have since been blacklisted.

More work needs to be done to provide a cleanup routine for existing projects that have since been blacklisted to return funds, mark project as disabled, etc. Someone with more domain knowledge should pick this up.

Ultimately this should be an opt-in system only, but that will require more time than I have to give. I hope this is a step in the right direction.

Fixes: #127, #145, #149, #150, #153, #155, #158, #159, #161, #162, #164, #166
Related: #146, #154, #157

@bill-auger bill-auger mentioned this pull request Nov 1, 2014
@bill-auger
Copy link
Contributor

this feature could also be extended to include a contributor blacklist

@admtnnr
Copy link
Contributor Author

admtnnr commented Nov 2, 2014

If you invert this you can, and probably should, use it as an opt-in system instead by maintaining a whitelist of the URLs. If projects want to be able to receive tips then they have to submit a PR to add their projects to the whitelist.

I'm happy to flip the logic to make it a whitelist, however, this will effectively remove all existing projects. I'll leave that decision up to the maintainers.

@bearbin
Copy link

bearbin commented Nov 2, 2014

I don't agree with this feature at all. Should a project maintainer have a say in what other members of the project decide to do with the money being offered to them?

It's much better to just have an easy method to unsubscribe from emails for individual contributors.

sashazykov added a commit that referenced this pull request Nov 2, 2014
Allow projects to be blacklisted for tips.
@sashazykov sashazykov merged commit f670df9 into tip4commit:master Nov 2, 2014
@arsenische
Copy link
Contributor

It's much better to just have an easy method to unsubscribe from emails for individual contributors.

@bearbin individual contributors always had this feature

though now it doesn't make sense since we don't send out emails anymore

@adamtanner , thanks for your contribution! (I personally think it is wrong to prevent free cooperation of people, but since project maintainers do really complain about it, we need to address it)

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.

Please remove mitsuhiko/*
5 participants