Skip to content

chore: ignore ImageSharp major updates in Dependabot#334

Merged
plz12345 merged 1 commit into
eros-developfrom
chore-ignore-imagesharp-major
Jul 20, 2026
Merged

chore: ignore ImageSharp major updates in Dependabot#334
plz12345 merged 1 commit into
eros-developfrom
chore-ignore-imagesharp-major

Conversation

@plz12345

Copy link
Copy Markdown
Contributor

Database Migration

NO

Description

SixLabors.ImageSharp 4.0.0 enforces a commercial licence at build time via an MSBuild target shipped in the package. Without a purchased key the build fails outright:

sixlabors.imagesharp/4.0.0/build/SixLabors.ImageSharp.targets(27,5):
  error : No Six Labors license found. Set $(SixLaborsLicenseKey), set $(SixLaborsLicenseFile),
          or add a 'sixlabors.lic' file to the project/workspace.
  error : Please obtain a license from https://sixlabors.com/pricing/

This is not an API break that can be coded around — it fails during build, so it took out all ten backend RID jobs in the grouped NuGet bump (#333), which in turn skipped unit_test, unit_test_postgres and integration_test.

This PR adds an ignore rule for semver-major updates to SixLabors.ImageSharp, keeping the grouped NuGet PRs on the 3.x line. 3.1.12 (already in tree) is the latest 3.x release; 4.0.0 is the only version above it, so there is no intermediate to move to. Without this rule the daily group PR keeps reintroducing 4.0.0 and the build stays red.

Also normalises the Dependabot commit-message.prefix for both ecosystems from Chore to lowercase chore, matching the conventional prefixes CONTRIBUTING already tolerates.

Follow-up, not in scope here: ImageSharp is used in exactly one place — a ~5-line JPEG thumbnail resize in src/NzbDrone.Core/MediaCover/ImageResizer.cs. Staying on 3.1.x means eventually losing security fixes, so replacing it is worth its own issue.

Screenshot(s) (if UI related)

N/A — CI configuration only.

Todos

  • Tests
  • Translation Keys (./src/NzbDrone.Core/Localization/Core/en.json)

Neither applies: this changes .github/dependabot.yml only, with no application code and no user-facing strings.

Issues Fixed or Closed by this PR

No tracked issue. Unblocks #333, which should be closed and left to regenerate once this merges.

SixLabors.ImageSharp 4.0.0 enforces a commercial licence at build time
via an MSBuild target, failing the restore with "No Six Labors license
found" unless a purchased key is supplied. This broke every backend RID
job in the grouped NuGet bump (#333).

Pin the grouped updates to the 3.x line by ignoring semver-major bumps
for ImageSharp so the daily group PR stops reintroducing 4.0.0.

Also normalise the Dependabot commit-message prefixes to lowercase
"chore" to match the conventional prefixes used elsewhere.
@sonarqubecloud

Copy link
Copy Markdown

@plz12345
plz12345 merged commit 4ca92df into eros-develop Jul 20, 2026
38 checks passed
@plz12345
plz12345 deleted the chore-ignore-imagesharp-major branch July 20, 2026 17:26
This was referenced Jul 20, 2026
plz12345 added a commit that referenced this pull request Jul 20, 2026
#### Database Migration

<!-- If you update database schema, the migration number must be noted
here -->

NO

#### Description

Rolling the 3.3.5 development cycle into a stable release.

Replaces #330, which was headed by `eros-develop` and therefore kept
absorbing
post-3.3.5 commits as they landed. This one is headed by the frozen
`release-3.3.5` branch, pinned to 39b1db6 — the exact commit carrying
the
`v3.3.5` marker tag — so the release contents can't drift.

Commits after the tag (#331, #334, #335) are deliberately left on
`eros-develop` for the next cycle.

**Merge with "Create a merge commit" — not squash, not rebase.** The
build
derives its base version from the `v3.3.5` tag's commit SHA being
reachable
from `eros`; squashing collapses that SHA and rebasing rewrites it,
either of
which makes the release build as 3.3.4.

#### Screenshot(s) (if UI related)

<details>
<!-- paste screenshots below here.-->

<!-- paste screenshots above here.  MAKE SURE THE ARE SFW -->
</details>

#### Todos

- [ ] Tests
- [ ] Translation Keys (./src/NzbDrone.Core/Localization/Core/en.json)

#### Issues Fixed or Closed by this PR

<!-- If multiple, put each on a separate line -->

<!-- Release roll-up; individual fixes were linked on their own PRs. -->
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.

1 participant