[PR #12578] [MERGED] Cleanup tasks #13264

Closed
opened 2026-02-07 07:09:54 +03:00 by OVERLORD · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/12578
Author: @Shadowghost
Created: 9/4/2024
Status: Merged
Merged: 9/6/2024
Merged by: @nielsvanvelzen

Base: masterHead: task-cleanup


📝 Commits (2)

📊 Changes

23 files changed (+187 additions, -258 deletions)

View changed files

📝 Emby.Server.Implementations/ScheduledTasks/ScheduledTaskWorker.cs (+20 -85)
📝 Emby.Server.Implementations/ScheduledTasks/TaskManager.cs (+16 -33)
📝 Emby.Server.Implementations/ScheduledTasks/Tasks/AudioNormalizationTask.cs (+1 -1)
📝 Emby.Server.Implementations/ScheduledTasks/Tasks/ChapterImagesTask.cs (+14 -14)
📝 Emby.Server.Implementations/ScheduledTasks/Tasks/CleanActivityLogTask.cs (+1 -2)
📝 Emby.Server.Implementations/ScheduledTasks/Tasks/CleanupCollectionAndPlaylistPathsTask.cs (+4 -4)
📝 Emby.Server.Implementations/ScheduledTasks/Tasks/DeleteCacheFileTask.cs (+4 -7)
📝 Emby.Server.Implementations/ScheduledTasks/Tasks/DeleteLogFileTask.cs (+7 -10)
📝 Emby.Server.Implementations/ScheduledTasks/Tasks/DeleteTranscodeFileTask.cs (+3 -3)
📝 Emby.Server.Implementations/ScheduledTasks/Tasks/OptimizeDatabaseTask.cs (+7 -10)
📝 Emby.Server.Implementations/ScheduledTasks/Tasks/PeopleValidationTask.cs (+10 -8)
📝 Emby.Server.Implementations/ScheduledTasks/Tasks/PluginUpdateTask.cs (+7 -9)
📝 Emby.Server.Implementations/ScheduledTasks/Tasks/RefreshMediaLibraryTask.cs (+1 -6)
📝 Emby.Server.Implementations/ScheduledTasks/Triggers/DailyTrigger.cs (+4 -16)
📝 Emby.Server.Implementations/ScheduledTasks/Triggers/StartupTrigger.cs (+5 -19)
📝 Emby.Server.Implementations/ScheduledTasks/Triggers/WeeklyTrigger.cs (+5 -17)
📝 MediaBrowser.Model/Tasks/IConfigurableScheduledTask.cs (+7 -2)
📝 MediaBrowser.Model/Tasks/IScheduledTaskWorker.cs (+2 -1)
📝 MediaBrowser.Model/Tasks/ITaskManager.cs (+28 -3)
📝 MediaBrowser.Model/Tasks/TaskCompletionEventArgs.cs (+16 -2)

...and 3 more files

📄 Description

  • Add docs
  • Use inheritdoc where possible
  • Switch some arrays to IReadOnlyList
  • Add .ConfigureAwait(false) to AudioNormalizationTask async LUFS calucaltion call

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/jellyfin/jellyfin/pull/12578 **Author:** [@Shadowghost](https://github.com/Shadowghost) **Created:** 9/4/2024 **Status:** ✅ Merged **Merged:** 9/6/2024 **Merged by:** [@nielsvanvelzen](https://github.com/nielsvanvelzen) **Base:** `master` ← **Head:** `task-cleanup` --- ### 📝 Commits (2) - [`08ed0a9`](https://github.com/jellyfin/jellyfin/commit/08ed0a9a5d2a241abe4304e03fab4ae74291e3c9) Cleanup tasks - [`994266a`](https://github.com/jellyfin/jellyfin/commit/994266a54f60d89efd4bc54580f2340d70baf59f) Fixup ### 📊 Changes **23 files changed** (+187 additions, -258 deletions) <details> <summary>View changed files</summary> 📝 `Emby.Server.Implementations/ScheduledTasks/ScheduledTaskWorker.cs` (+20 -85) 📝 `Emby.Server.Implementations/ScheduledTasks/TaskManager.cs` (+16 -33) 📝 `Emby.Server.Implementations/ScheduledTasks/Tasks/AudioNormalizationTask.cs` (+1 -1) 📝 `Emby.Server.Implementations/ScheduledTasks/Tasks/ChapterImagesTask.cs` (+14 -14) 📝 `Emby.Server.Implementations/ScheduledTasks/Tasks/CleanActivityLogTask.cs` (+1 -2) 📝 `Emby.Server.Implementations/ScheduledTasks/Tasks/CleanupCollectionAndPlaylistPathsTask.cs` (+4 -4) 📝 `Emby.Server.Implementations/ScheduledTasks/Tasks/DeleteCacheFileTask.cs` (+4 -7) 📝 `Emby.Server.Implementations/ScheduledTasks/Tasks/DeleteLogFileTask.cs` (+7 -10) 📝 `Emby.Server.Implementations/ScheduledTasks/Tasks/DeleteTranscodeFileTask.cs` (+3 -3) 📝 `Emby.Server.Implementations/ScheduledTasks/Tasks/OptimizeDatabaseTask.cs` (+7 -10) 📝 `Emby.Server.Implementations/ScheduledTasks/Tasks/PeopleValidationTask.cs` (+10 -8) 📝 `Emby.Server.Implementations/ScheduledTasks/Tasks/PluginUpdateTask.cs` (+7 -9) 📝 `Emby.Server.Implementations/ScheduledTasks/Tasks/RefreshMediaLibraryTask.cs` (+1 -6) 📝 `Emby.Server.Implementations/ScheduledTasks/Triggers/DailyTrigger.cs` (+4 -16) 📝 `Emby.Server.Implementations/ScheduledTasks/Triggers/StartupTrigger.cs` (+5 -19) 📝 `Emby.Server.Implementations/ScheduledTasks/Triggers/WeeklyTrigger.cs` (+5 -17) 📝 `MediaBrowser.Model/Tasks/IConfigurableScheduledTask.cs` (+7 -2) 📝 `MediaBrowser.Model/Tasks/IScheduledTaskWorker.cs` (+2 -1) 📝 `MediaBrowser.Model/Tasks/ITaskManager.cs` (+28 -3) 📝 `MediaBrowser.Model/Tasks/TaskCompletionEventArgs.cs` (+16 -2) _...and 3 more files_ </details> ### 📄 Description * Add docs * Use `inheritdoc` where possible * Switch some arrays to `IReadOnlyList` * Add `.ConfigureAwait(false)` to `AudioNormalizationTask` async LUFS calucaltion call --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
OVERLORD added the pull-request label 2026-02-07 07:09:54 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/jellyfin#13264