carvel-dev / carvel-dev/imgpkg

onboarding improvements

Open
#237 0 comments 0 reactions 1 assignee Claimed by @DennisDenuto View on GitHub
carvel accepted priority/unprioritized-backlog
Dominant language
Go
Stars
284
Forks
70
PR merge metrics
No merged PRs in 30d

Description

Worked with @seemiller

we have identified the following areas of improvement to help folk onboard and make contributions to imgpkg:

- [x] ui vs logger -> hard to make changes since there are 2 different logging mechanisms. https://github.com/vmware-tanzu/carvel-imgpkg/pull/259
- [x] prefixlogger vs levellogger in copy.go (2 different loggers used)
- [ ] Document the different hack/test scripts
- [ ] Have a friendly way to choose which test suites to run
- [ ] Add documentation around when to use a fake registry vs a real registry within a test
- [ ] Add documentation for folk using intellij / goland on how to run E2E tests to run locally (Setting env variables such as the location of the binary and the other E2E envs
- [ ] use docker golang image to build and run tests locally

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.