chore: update specterops nuget source to shcommon-nuget BED-8241#302
Conversation
Walkthroughsleet.json S3 feed source updated to use the shcommon-nuget bucket/path; publish workflow prune step comment clarified and its package index URL changed to read sleet.packageindex.json from shcommon-nuget. ChangesS3 Feed & Workflow Update
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Tip 💬 Introducing Slack Agent: The best way for teams to turn conversations into code.Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.
Built for teams:
One agent for your entire SDLC. Right inside Slack. 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: 2
🤖 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 @.github/workflows/publish-dev-package.yml:
- Around line 6-7: Restore the "v4" push trigger in the workflow's push branch
list so the job runs on pushes to the release branch; update the array that
currently contains "lfalslev/bed-8241" (and has "v4" commented out) to include
"v4" again alongside any feature branches, ensuring the workflow will run for
normal v4 pushes after merge.
- Line 59: The workflow hardcodes $dryRun = $true which prevents prune from ever
running; change this to read a workflow input or environment variable (e.g.,
inputs.prune_dry_run or PRUNE_DRY_RUN) and default to false so pruning runs by
default, then set $dryRun = ${{ github.event.inputs.prune_dry_run ||
env.PRUNE_DRY_RUN }} (or equivalent expression) and update the workflow
inputs/env to document the flag so debugging can explicitly enable dry-run mode;
ensure any action/step that uses $dryRun (the variable named $dryRun) respects
the parsed boolean value.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 22e08e58-91a0-4510-aca9-728e89d3e435
📒 Files selected for processing (2)
.github/workflows/publish-dev-package.ymlsleet.json
Description
Old bloodhound-ad s3 bucket has been replaced with new shcommon-nuget bucket for the specterops nuget feed. This updates sleet config and github workflows to use new the new feed.
Motivation and Context
This PR addresses: BED-8241
How Has This Been Tested?
Ran
sleet validateand tested publishing and pruning -dev packages to feed.Screenshots (if appropriate):
Types of changes
Checklist:
Summary by CodeRabbit