Update boilerplate to latest version - #345
Conversation
Update boilerplate framework with agentic SDLC check improvements and operator name extraction fix. Changes include: - Added `onError: continue` to check-file-existence and check-content-validation Tekton pipeline tasks so non-fatal check failures no longer block the pipeline - Improved CLAUDE.md content validation to resolve one level of @ include references before scanning for required sections - Fixed OPERATOR_NAME extraction regex in the update script to handle assignment operators (= or :=) between the constant name and value - Updated boilerplate commit hash to 263afb10321f05af42d3fc928b7ccffd601d5996 Changes applied via `make boilerplate-update`. Co-Authored-By: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository YAML (base), Central YAML (inherited) Review profile: CHILL Plan: Enterprise Run ID: ⛔ Files ignored due to path filters (3)
📒 Files selected for processing (1)
WalkthroughThis PR modifies a Tekton PipelineRun YAML for the deadmanssnitch-operator agentic SDLC pull request check. It adds ChangesPipeline Task Error Handling and Content Validation
Estimated code review effort: 1 (Trivial) | ~5 minutes Possibly related PRs
Suggested labels: Suggested reviewers: 🚥 Pre-merge checks | ✅ 15✅ Passed checks (15 passed)
✨ 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 |
|
@anispate: all tests passed! Full PR test history. Your PR dashboard. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #345 +/- ##
=======================================
Coverage 43.28% 43.28%
=======================================
Files 11 11
Lines 834 834
=======================================
Hits 361 361
Misses 424 424
Partials 49 49 🚀 New features to boost your workflow:
|
|
/lgtm |
1 similar comment
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: aliceh, anispate, nephomaniac The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Summary
onError: continueto Tekton pipeline tasks so non-fatal agentic SDLC check failures no longer block the pipeline@include references before scanning for required sectionsOPERATOR_NAMEextraction regex in the boilerplate update script to handle assignment operators (=or:=) between the constant name and valueChanges
.tekton/deadmanssnitch-operator-agentic-sdlc-check-pull-request.yaml: AddedonError: continuetocheck-file-existenceandcheck-content-validationtasksboilerplate/openshift/golang-osd-operator/agentic-sdlc-check-pull-request.yaml.tmpl: Same template-level changeboilerplate/openshift/golang-osd-operator/update: FixedOPERATOR_NAMEsed regex to matchOperatorName = "..."andOperatorName := "..."patternsboilerplate/_data/last-boilerplate-commit: Updated to263afb10321f05af42d3fc928b7ccffd601d5996Validation
make boilerplate-updateReferences
Created with assistance from Claude claude@anthropic.com
Created using the rosa-sre-operator-maintenance plugin from rosa-claude-plugins
Summary by CodeRabbit