Skip to content

refactor: move service name extraction utilities to shared helpers#10

Merged
Kiran01bm merged 4 commits intoblock:mainfrom
Kiran01bm:kiran01bm-refactor-1
Apr 15, 2026
Merged

refactor: move service name extraction utilities to shared helpers#10
Kiran01bm merged 4 commits intoblock:mainfrom
Kiran01bm:kiran01bm-refactor-1

Conversation

@Kiran01bm
Copy link
Copy Markdown
Contributor

Summary

Refactors service name extraction utilities from aurora.go into shared helper functions in pkg/inventory/wiz/helpers.go to eliminate code duplication and enable reuse across all inventory sources.

@Kiran01bm Kiran01bm requested a review from bakayolo as a code owner April 14, 2026 07:54
@Kiran01bm Kiran01bm force-pushed the kiran01bm-refactor-1 branch from 9fd5f1a to 184a1ff Compare April 15, 2026 00:11
Kiran01bm and others added 2 commits April 15, 2026 10:14
Co-authored-by: Claude Code <noreply@anthropic.com>
Ai-assisted: true
Co-authored-by: Claude Code <noreply@anthropic.com>
Ai-assisted: true
@Kiran01bm Kiran01bm force-pushed the kiran01bm-refactor-1 branch from 184a1ff to f03912d Compare April 15, 2026 00:15
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 15, 2026

Codecov Report

❌ Patch coverage is 92.59259% with 2 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
pkg/inventory/wiz/helpers.go 92.59% 1 Missing and 1 partial ⚠️
Files with missing lines Coverage Δ
pkg/inventory/wiz/aurora.go 84.00% <ø> (ø)
pkg/inventory/wiz/helpers.go 78.09% <92.59%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

bakayolo
bakayolo previously approved these changes Apr 15, 2026
bakayolo
bakayolo previously approved these changes Apr 15, 2026
@Kiran01bm Kiran01bm merged commit 550f2fb into block:main Apr 15, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants