soldeer: add .soldeerignore (unblock autopublish push)#25
Conversation
…tive-file prompt forge soldeer push flags repo dotfiles (.git/.github/etc.) as sensitive and prompts for confirmation; in CI there's no stdin so it fails with 'error during IO operation: not connected'. Mirror raindex: publish only src/.
|
Warning Review limit reached
More reviews will be available in 53 minutes and 55 seconds. Learn how PR review limits work. Your organization has run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After more reviews become available, a review can be triggered using the 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 include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available. Please see our Fair Usage Limits Policy for further information. WalkthroughA ChangesSoldeer Package Configuration
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In @.soldeerignore:
- Around line 1-5: Add REUSE v3.3 SPDX headers to the top of .soldeerignore by
inserting SPDX-FileCopyrightText and SPDX-License-Identifier header lines (with
the appropriate copyright holder and license identifier used across this repo)
so the file becomes REUSE-compliant; update the top of the .soldeerignore file
to include those two SPDX tags before any comments or content to unblock CI
legal checks.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
|
Reviewed 871ebbe: APPROVE — add .soldeerignore (mirror raindex) so the autopublish soldeer push doesn't hang on the sensitive-file prompt in CI. Verified via dry-run: package trims to src/ + LICENSE(S) + README only, no dotfiles. Merging re-triggers package-release to publish rain-pyth~0.1.0. |
|
Reviewed 1cc84fe: APPROVE — .soldeerignore (mirror raindex) trims publish to src/+license/readme so the CI soldeer push doesn't hang on the sensitive-file prompt; SPDX header added for reuse lint. Merging re-triggers package-release to publish rain-pyth~0.1.0. |
rain-pyth's first soldeer publish hangs because
forge soldeer pushflags repo dotfiles as sensitive and prompts for confirmation — CI has no stdin, so it errorsduring IO operation: not connected. Mirrors raindex's.soldeerignore(publish onlysrc/+ license/readme). Merging this re-triggers package-release, which should then publishrain-pyth~0.1.0. Towards #22.Summary by CodeRabbit