Skip to content

Comments

fix(ci): update to net8.0 only for Lidarr 3.x compatibility#338

Closed
RicherTunes wants to merge 1 commit intomainfrom
fix/net8-only-nightly
Closed

fix(ci): update to net8.0 only for Lidarr 3.x compatibility#338
RicherTunes wants to merge 1 commit intomainfrom
fix/net8-only-nightly

Conversation

@RicherTunes
Copy link
Owner

Summary

  • Change TargetFrameworks from net6.0;net8.0 to net8.0 only
  • Remove 6.0.x from nightly CI matrix
  • Clean up net6.0 LidarrPath fallback logic

Problem

Nightly CI was failing with:

error NU1201: Project Lidarr.Plugin.Common is not compatible with net6.0
(.NETCoreApp,Version=v6.0). Project Lidarr.Plugin.Common supports:
net8.0 (.NETCoreApp,Version=v8.0)

The lidarr.plugin.common submodule only targets net8.0, causing restore failures when brainarr tried to build for net6.0.

Solution

Align brainarr with Lidarr 3.x (mainline) which requires .NET 8.0. Remove all net6.0 targeting and fallback paths.

Test plan

  • dotnet restore succeeds
  • Local build passes
  • CI nightly build passes

🤖 Generated with Claude Code

- Change TargetFrameworks from net6.0;net8.0 to net8.0 only
- Remove net6.0 from nightly CI matrix
- Clean up net6.0 LidarrPath fallback logic
- Align with lidarr.plugin.common which only supports net8.0

This fixes nightly CI failures caused by TFM incompatibility with the
lidarr.plugin.common submodule.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@github-actions
Copy link
Contributor

⚠️ Deprecation Warning: The deny-licenses option is deprecated for possible removal in the next major release. For more information, see issue 997.

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@RicherTunes
Copy link
Owner Author

Stale — superseded by packaging-gates adoption (#397) and ecosystem standardization.

@RicherTunes RicherTunes closed this Feb 6, 2026
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