Skip to content

refactor: remove helmchart installation and update logic#167

Merged
matteogastaldello merged 11 commits intomainfrom
166-helm-library-integration
Feb 16, 2026
Merged

refactor: remove helmchart installation and update logic#167
matteogastaldello merged 11 commits intomainfrom
166-helm-library-integration

Conversation

@matteogastaldello
Copy link
Collaborator

  • Deleted the install.go and update.go files along with their associated test files.
  • Removed the postrenderer.go file which handled post-rendering of Helm charts.
  • Introduced a new processor package to handle release decoding and hashing.
  • Added processor tests and benchmarks for performance evaluation.
  • Implemented a tracer for HTTP requests to log resource interactions.
  • Updated main.go to integrate the new dynamic tools and REST mapper.

- Deleted the install.go and update.go files along with their associated test files.
- Removed the postrenderer.go file which handled post-rendering of Helm charts.
- Introduced a new processor package to handle release decoding and hashing.
- Added processor tests and benchmarks for performance evaluation.
- Implemented a tracer for HTTP requests to log resource interactions.
- Updated main.go to integrate the new dynamic tools and REST mapper.
@matteogastaldello matteogastaldello linked an issue Feb 6, 2026 that may be closed by this pull request
@matteogastaldello matteogastaldello self-assigned this Feb 13, 2026
@codecov
Copy link

codecov bot commented Feb 13, 2026

Codecov Report

❌ Patch coverage is 62.19512% with 31 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
internal/tools/processor/types.go 12.50% 14 Missing ⚠️
internal/tools/processor/processor.go 75.75% 6 Missing and 2 partials ⚠️
internal/tools/hasher/hasher.go 0.00% 7 Missing ⚠️
internal/tools/dynamic/dynamic.go 90.90% 1 Missing and 1 partial ⚠️

📢 Thoughts on this report? Let us know!

@matteogastaldello matteogastaldello merged commit 7a4c1ac into main Feb 16, 2026
4 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.

Helm Library Integration

1 participant