Skip to content
This repository was archived by the owner on Nov 12, 2025. It is now read-only.

Remove deprecated codenotary fields#60

Merged
frenck merged 1 commit into
mainfrom
frenck-2025-0479
Nov 12, 2025
Merged

Remove deprecated codenotary fields#60
frenck merged 1 commit into
mainfrom
frenck-2025-0479

Conversation

@frenck

@frenck frenck commented Nov 12, 2025

Copy link
Copy Markdown
Member

Proposed Changes

SSIA

Summary by CodeRabbit

  • Chores
    • Removed code verification configuration from Readarr addon.

@frenck frenck added the bugfix Inconsistencies or issues which will cause a problem for users or implementors. label Nov 12, 2025
@coderabbitai

coderabbitai Bot commented Nov 12, 2025

Copy link
Copy Markdown

Walkthrough

Removed codenotary configuration metadata from readarr addon files, eliminating signing and base image references from both build and configuration specifications without altering remaining addon structure or functionality.

Changes

Cohort / File(s) Summary
Codenotary Configuration Removal
readarr/build.yaml, readarr/config.yaml
Removed codenotary configuration block from build.yaml (base_image and signer entries) and removed codenotary property from config.yaml

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

  • Both changes are straightforward removal of configuration entries with no impact on remaining config structure
  • No logic changes or structural modifications required

Poem

🐰 Hop, hop—the codenotary goes away,
No more signatures to sign the day!
Readarr hops free, configuration light,
Cleaner config makes everything right!

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Remove deprecated codenotary fields' clearly and specifically summarizes the main change: removing codenotary configuration from both readarr/build.yaml and readarr/config.yaml files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch frenck-2025-0479

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 2f2d5b3 and 4136018.

📒 Files selected for processing (2)
  • readarr/build.yaml (0 hunks)
  • readarr/config.yaml (0 hunks)
💤 Files with no reviewable changes (2)
  • readarr/build.yaml
  • readarr/config.yaml

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@frenck frenck merged commit cdd4d0a into main Nov 12, 2025
22 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

bugfix Inconsistencies or issues which will cause a problem for users or implementors.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant