Skip to content

chore(deps): limit dependencies in favor of built-ins or core libs#4901

Open
soltysh wants to merge 2 commits into
zarf-dev:mainfrom
soltysh:deps_replacement_1
Open

chore(deps): limit dependencies in favor of built-ins or core libs#4901
soltysh wants to merge 2 commits into
zarf-dev:mainfrom
soltysh:deps_replacement_1

Conversation

@soltysh
Copy link
Copy Markdown
Contributor

@soltysh soltysh commented May 14, 2026

Description

This PR introduces two changes:

  1. Replace controller-runtime helpers with k8s native client-go ones. Allowing us to drop direct dependency, although not entirely eliminating it due to transitive dependency through flux.
  2. Replace custom helpers (helpers.RemoveMatches) with golang buitl-in slices counterparts.

Related Issue

Relates to #4895

Checklist before merging

soltysh added 2 commits May 14, 2026 15:34
…e for mapper

Signed-off-by: Maciej Szulik <maciej@defenseunicorns.com>
Signed-off-by: Maciej Szulik <maciej@defenseunicorns.com>
@soltysh soltysh requested review from a team as code owners May 14, 2026 14:35
@netlify
Copy link
Copy Markdown

netlify Bot commented May 14, 2026

Deploy Preview for zarf-docs canceled.

Name Link
🔨 Latest commit 4ca3ca7
🔍 Latest deploy log https://app.netlify.com/projects/zarf-docs/deploys/6a05dda325e1d70008dd8404

@codecov
Copy link
Copy Markdown

codecov Bot commented May 14, 2026

Codecov Report

❌ Patch coverage is 20.00000% with 4 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/pkg/cluster/cluster.go 0.00% 2 Missing ⚠️
src/pkg/packager/remove.go 0.00% 2 Missing ⚠️
Files with missing lines Coverage Δ
src/pkg/packager/pull.go 50.00% <100.00%> (ø)
src/pkg/cluster/cluster.go 49.77% <0.00%> (+0.33%) ⬆️
src/pkg/packager/remove.go 0.00% <0.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

1 participant