chore: use official artifacthub stepactions for git-clone#2767
chore: use official artifacthub stepactions for git-clone#2767theakshaypant wants to merge 2 commits into
Conversation
Preserves the error chain for errors.Is/errors.As callers. Co-Authored-By: Claude <noreply@anthropic.com>
There was a problem hiding this comment.
Code Review
This pull request updates Tekton pipeline configurations to use the Hub resolver for the git-clone stepaction (version 1.7.0) instead of the HTTP resolver, and removes the local stepaction definition. Additionally, it improves error handling in bootstrap.go by wrapping the error with %w. Feedback was provided regarding a version mismatch in the test data, which still references version 1.6.0 of the stepaction instead of 1.7.0.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
Replaced the local HTTP-based git-clone stepaction with the official Tekton Hub resolver across Tekton workflows. Removed the redundant local stepaction definition file to keep configuration centralized. Signed-off-by: Chmouel Boudjnah <chmouel@redhat.com>
a9a1bdd to
b54f7be
Compare
|
/test |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2767 +/- ##
=======================================
Coverage 59.73% 59.73%
=======================================
Files 210 210
Lines 21112 21112
=======================================
Hits 12611 12611
Misses 7706 7706
Partials 795 795 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
📝 Description of the Change
Replace the vendored
git-cloneStepAction (.tekton/stepactions/git-clone.yaml) with a reference to the upstreamtektoncdcatalog via thehubresolver. The local copy was a verbatim duplicate of catalog version 0.2 — no reason to carry it.Also fixes error wrapping in
DetectPacInstallation(%s+.Error()→%w) to preserve the error chain forerrors.Is/errors.As.Clone of #2758 with the changes requested in #2758 (review)
🔗 Linked GitHub Issue
Fixes #
🧪 Testing Strategy
🤖 AI Assistance
✅ Submitter Checklist
fix:,feat:) matches the "Type of Change" I selected above.make testandmake lintlocally to check for and fix any issues. For an efficient workflow, I have considered installing pre-commit and runningpre-commit installto automate these checks.