Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[charts-build-scripts] technical debt of charts-build-scripts #438

Open
6 tasks
nicholasSUSE opened this issue Jul 3, 2024 · 0 comments
Open
6 tasks

[charts-build-scripts] technical debt of charts-build-scripts #438

nicholasSUSE opened this issue Jul 3, 2024 · 0 comments
Assignees

Comments

@nicholasSUSE
Copy link
Contributor

Scope

The charts-build-scripts repo is now owned by the release team.

It is an ancient and legacy code with few portions of new code.

This repository handles 2 specific responsibilities across several features:

  • Charts generation and development work
  • Charts release process

The release-team has higher standards of go code quality therefor the charts-build-scripts must be improved.

Tasks

  • Remove the pkg directory
  • Scope lock all err values
  • Use 0 value declaration
  • Unexport all exposed symbols that don't need to be
  • Condense the packages down to what is necessarily needed for APIs
  • Update file names to conform with standards
@nicholasSUSE nicholasSUSE self-assigned this Jul 23, 2024
@nicholasSUSE nicholasSUSE changed the title technical debt of charts-build-scripts [charts-build-scripts] technical debt of charts-build-scripts Oct 24, 2024
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

No branches or pull requests

1 participant