[PR #13444] [MERGED] Fix spelling #13581

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

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/13444
Author: @jsoref
Created: 1/27/2025
Status: Merged
Merged: 1/28/2025
Merged by: @Bond-009

Base: masterHead: spelling


📝 Commits (10+)

📊 Changes

18 files changed (+23 additions, -23 deletions)

View changed files

📝 Emby.Naming/TV/SeriesResolver.cs (+1 -1)
📝 Jellyfin.Api/Controllers/AudioController.cs (+1 -1)
📝 Jellyfin.Api/Helpers/StreamingHelpers.cs (+1 -1)
📝 Jellyfin.Data/Entities/BaseItemMetadataField.cs (+1 -1)
📝 Jellyfin.Data/Entities/BaseItemTrailerType.cs (+1 -1)
📝 Jellyfin.Data/Entities/ItemValue.cs (+1 -1)
📝 Jellyfin.Data/Entities/PeopleBaseItemMap.cs (+1 -1)
📝 Jellyfin.Server.Implementations/Item/BaseItemRepository.cs (+5 -5)
📝 MediaBrowser.Controller/MediaEncoding/EncodingHelper.cs (+1 -1)
📝 MediaBrowser.Controller/Session/ISessionManager.cs (+1 -1)
📝 MediaBrowser.Controller/Session/SessionInfo.cs (+1 -1)
📝 MediaBrowser.LocalMetadata/Savers/BaseXmlSaver.cs (+1 -1)
📝 MediaBrowser.Model/Dto/SessionInfoDto.cs (+1 -1)
📝 MediaBrowser.Providers/MediaInfo/FFProbeVideoInfo.cs (+1 -1)
📝 src/Jellyfin.Drawing.Skia/SplashscreenBuilder.cs (+1 -1)
📝 src/Jellyfin.LiveTv/Listings/SchedulesDirectDtos/LineupDto.cs (+1 -1)
📝 src/Jellyfin.LiveTv/Listings/SchedulesDirectDtos/MetadataDto.cs (+1 -1)
📝 src/Jellyfin.LiveTv/TunerHosts/M3uParser.cs (+2 -2)

📄 Description

Changes
Fixes misspellings identified by the check-spelling action.

The misspellings have been reported at https://github.com/jsoref/jellyfin/actions/runs/12979401882#summary-36195047679

The action will report that the changes in this PR would make it happy: https://github.com/jsoref/jellyfin/actions/runs/12979402012#summary-36195047936 -- Note that there are a number of tests that appear to be consistently failing. In order to get CI not not fail miserably, I included 0c236225278edacc5f9c93467d628ced8a960660 in my test PR (but they aren't included in this PR).


As before, if people are happy with the changes, I'll squash them.

There's one change that I omitted which was BluRay to Bluray (for everything except file names) because that upset the API checker. Ignoring files with BluRay in their names, there's an exactly equal number of BluRay and Bluray, I looked around and checked with people and do believe that the latter would be more correct (although it'd obviously require adding a new set of APIs and deprecating the old ones). If people are interested, I'm happy to make a PR for that separately.


🔄 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/13444 **Author:** [@jsoref](https://github.com/jsoref) **Created:** 1/27/2025 **Status:** ✅ Merged **Merged:** 1/28/2025 **Merged by:** [@Bond-009](https://github.com/Bond-009) **Base:** `master` ← **Head:** `spelling` --- ### 📝 Commits (10+) - [`863e321`](https://github.com/jellyfin/jellyfin/commit/863e321e10e4ef4ef763591766c3c18f333387a8) spelling: anamorphic - [`1e9e277`](https://github.com/jellyfin/jellyfin/commit/1e9e27731bbb63b0e9a5a66a57294d28dda96eb2) spelling: associated - [`06521e1`](https://github.com/jellyfin/jellyfin/commit/06521e17675faa850a7848a1a78ea1810f147e8c) spelling: channelinfo - [`3492331`](https://github.com/jellyfin/jellyfin/commit/3492331c673d2e9d60e2a74e0214ebf300bb2352) spelling: eagerly - [`691e810`](https://github.com/jellyfin/jellyfin/commit/691e8107548f46b6d85296e265f7649c48c6b660) spelling: enumerable - [`8a60634`](https://github.com/jellyfin/jellyfin/commit/8a6063402bbd04bc780007110928fad5a8787f58) spelling: greater than/less than - [`5adf365`](https://github.com/jellyfin/jellyfin/commit/5adf365a47b09f3610c4a7d2656a5d9585c10a86) spelling: greater - [`c8ed159`](https://github.com/jellyfin/jellyfin/commit/c8ed159bd964837f930943e383d574ca4c32ce7b) spelling: lineup - [`d9e5fb9`](https://github.com/jellyfin/jellyfin/commit/d9e5fb904f1994b36709e4acfdf91da34bda07b6) spelling: logs out - [`cdd37aa`](https://github.com/jellyfin/jellyfin/commit/cdd37aa3caa9755d6eade7758b1dc7e453a52e67) spelling: names ### 📊 Changes **18 files changed** (+23 additions, -23 deletions) <details> <summary>View changed files</summary> 📝 `Emby.Naming/TV/SeriesResolver.cs` (+1 -1) 📝 `Jellyfin.Api/Controllers/AudioController.cs` (+1 -1) 📝 `Jellyfin.Api/Helpers/StreamingHelpers.cs` (+1 -1) 📝 `Jellyfin.Data/Entities/BaseItemMetadataField.cs` (+1 -1) 📝 `Jellyfin.Data/Entities/BaseItemTrailerType.cs` (+1 -1) 📝 `Jellyfin.Data/Entities/ItemValue.cs` (+1 -1) 📝 `Jellyfin.Data/Entities/PeopleBaseItemMap.cs` (+1 -1) 📝 `Jellyfin.Server.Implementations/Item/BaseItemRepository.cs` (+5 -5) 📝 `MediaBrowser.Controller/MediaEncoding/EncodingHelper.cs` (+1 -1) 📝 `MediaBrowser.Controller/Session/ISessionManager.cs` (+1 -1) 📝 `MediaBrowser.Controller/Session/SessionInfo.cs` (+1 -1) 📝 `MediaBrowser.LocalMetadata/Savers/BaseXmlSaver.cs` (+1 -1) 📝 `MediaBrowser.Model/Dto/SessionInfoDto.cs` (+1 -1) 📝 `MediaBrowser.Providers/MediaInfo/FFProbeVideoInfo.cs` (+1 -1) 📝 `src/Jellyfin.Drawing.Skia/SplashscreenBuilder.cs` (+1 -1) 📝 `src/Jellyfin.LiveTv/Listings/SchedulesDirectDtos/LineupDto.cs` (+1 -1) 📝 `src/Jellyfin.LiveTv/Listings/SchedulesDirectDtos/MetadataDto.cs` (+1 -1) 📝 `src/Jellyfin.LiveTv/TunerHosts/M3uParser.cs` (+2 -2) </details> ### 📄 Description **Changes** Fixes misspellings identified by the [check-spelling action](https://github.com/marketplace/actions/check-spelling). The misspellings have been reported at https://github.com/jsoref/jellyfin/actions/runs/12979401882#summary-36195047679 The action will report that the changes in this PR would make it happy: https://github.com/jsoref/jellyfin/actions/runs/12979402012#summary-36195047936 -- Note that there are a number of tests that appear to be consistently failing. In order to get CI not not fail miserably, I included 0c236225278edacc5f9c93467d628ced8a960660 in my test PR (but they aren't included in this PR). --- As before, if people are happy with the changes, I'll squash them. There's one change that I omitted which was `BluRay` to `Bluray` (for everything except file names) because that upset the API checker. Ignoring files with `BluRay` in their names, there's an exactly equal number of `BluRay` and `Bluray`, I looked around and checked with people and do believe that the latter would be more correct (although it'd obviously require adding a new set of APIs and deprecating the old ones). If people are interested, I'm happy to make a PR for that separately. --- <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:15:32 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/jellyfin#13581