Skip to content

chore(style): bulk shfmt v3.13.1 reformat across all scripts#39

Merged
bakerboy448 merged 1 commit into
mainfrom
chore-shfmt-v3.13.1-bulk-reformat
May 11, 2026
Merged

chore(style): bulk shfmt v3.13.1 reformat across all scripts#39
bakerboy448 merged 1 commit into
mainfrom
chore-shfmt-v3.13.1-bulk-reformat

Conversation

@bakerboy448
Copy link
Copy Markdown
Collaborator

Summary

  • Restores Pre-commit CI to green. Repo's main has been failing since 2026-04-18 because pre-commit-config pinned shfmt v3.13.1-1, but the repo files were formatted by an older shfmt version with different rules.
  • Pure formatting changes — no semantic effects.

Changes

  • 4-space → 2-space indent
  • pipe-at-EOL → \ continuation with pipe-at-BOL (--binary-next-line flag)
  • minor whitespace normalisation

12 .sh files touched; shellcheck --severity=warning clean across all.

Test plan

  • Pre-commit CI passes (it's been red for 3 weeks; this should fix it)
  • shfmt --diff is empty repo-wide after merge
  • No behavioral changes (formatting only)

Notes

This unblocks #38 (qbm-qbit + notifiarr-branch-builder cherry-pick), which inherits the broken Pre-commit run from main.

Restores Pre-commit CI to green. Repo's main has been failing since
2026-04-18 (shfmt v3.13.1 in pre-commit-config has stricter formatting
than the version that originally formatted these files).

Pure formatting — no semantic changes:
- 4-space → 2-space indent
- pipe-at-EOL → \\ continuation with pipe-at-BOL (--binary-next-line)
- minor whitespace normalisation

12 .sh files reformatted; shellcheck --severity=warning clean.

Co-Authored-By: Claude <[email protected]>
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 11, 2026

Warning

Rate limit exceeded

@bakerboy448 has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 43 minutes and 14 seconds before requesting another review.

You’ve run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 0fc39965-f1bb-4ae1-9435-dc439c33a255

📥 Commits

Reviewing files that changed from the base of the PR and between 77a0845 and f2f0ed2.

📒 Files selected for processing (12)
  • dupe.sh
  • f2b-dump.sh
  • omegabrr_upgrade.sh
  • pic-update.sh
  • pmm-update.sh
  • qbm-api-trigger.sh
  • qbm-update.sh
  • qui-xseed.sh
  • radarr_dupefinder.sh
  • servarr/servarr_bot_merge.sh
  • sonarr_dupefinder.sh
  • zfsburn.sh
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore-shfmt-v3.13.1-bulk-reformat

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.

@bakerboy448 bakerboy448 merged commit bb8759e into main May 11, 2026
3 checks passed
@bakerboy448 bakerboy448 deleted the chore-shfmt-v3.13.1-bulk-reformat branch May 11, 2026 01:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant