-
Notifications
You must be signed in to change notification settings - Fork 983
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
Soft Deletes Meta Issue #6091
Comments
Is this issue okay to use for feedback/input? Otherwise, I'll happily delete and post elsewhere. Here is a scenario that might cause frustrations in the future, as the pattern might propagate: Bots (like dependabot) + Pipenv.lock + package deletion = 😖 In this scenario, bots create updates (PRs) for Pipenv.lock that are hard for humans to read, they quickly and automatically pass the build, and once they are merged, they're broken -- because the package take a while for a human to delete, and PRs take a while for humans to merge. There are some ingredients in the deletion that can perhaps be put to use?
|
Hi @benjaoming, thanks for the feedback. I think that this issue would be resolved by completing #5837 and implementing "yanking" as an alternative to performing a deletion (regardless of whether it's a hard delete or a soft delete), as you would still be able to pin to a yanked package without issue. |
https://github.com/flipbit03/sqlalchemy-easy-softdelete seems to be a potential option for implementing soft-deletes. |
Adding related conversation: https://discuss.python.org/t/stop-allowing-deleting-things-from-pypi/17227 |
More xreffing: https://discuss.python.org/t/pep-763-limiting-deletions-on-pypi/69487 and PEP 763 for limiting deletions on PyPI (i.e. moving 100% to yanks/soft deletes). |
Currently, all deletes are good and proper Deletions.
This imposes quite a bit of risk for moderator activities, causes confusion and frustration for maintainers, and blocks us from implementing some important features down the line.
This issue is a "Meta Issue" for tracking progress on soft deletion implementation for models for which it makes sense.
Currently:
The text was updated successfully, but these errors were encountered: