Clean up old/unneeded mix tasks #470

Open
opened 2023-02-17 20:40:42 +00:00 by ilja · 0 comments
Contributor

See https://meta.akkoma.dev/t/frequency-to-run-update-users-following-task/366

This is about a mix task that was added 4 years ago for a one-time clean-up. There may be others and it's not always properly documented.

What I think we should do

  • Remove the one-time mix tasks. They wont be relevant any more, and if they are, they should probably be a migration instead.
    • Maybe note them in the changelog extra steps so that people with an instance that's old enough for this task to maybe have an impact, can choose to run it before upgrading. It needs to list each removed task + the date the task was introduced.
  • See if the ones who remain maybe need some better documentation on what they do and why they are there.

For this one I checked git blame, then looked up the commit on the pleroma gitlab and checked the MR there.

See https://meta.akkoma.dev/t/frequency-to-run-update-users-following-task/366 This is about a mix task that was added 4 years ago for a one-time clean-up. There may be others and it's not always properly documented. What I think we should do * [ ] Remove the one-time mix tasks. They wont be relevant any more, and if they are, they should probably be a migration instead. * Maybe note them in the changelog extra steps so that people with an instance that's old enough for this task to maybe have an impact, can choose to run it before upgrading. It needs to list each removed task + the date the task was introduced. * [ ] See if the ones who remain maybe need some better documentation on what they do and why they are there. For this one I checked git blame, then looked up the commit on the pleroma gitlab and checked the MR there.
Sign in to join this conversation.
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: AkkomaGang/akkoma#470
No description provided.