-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Add DEVELOPMENT.md #238
Add DEVELOPMENT.md #238
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a couple comments, thanks for adding this!
DEVELOPMENT.md
Outdated
|
||
#### Integration tests | ||
|
||
The integration tests live in [`integration`](./integration) and can be run with: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Our integration tests require having access to the kaniko-test project to run locally, so instead let's say that maintainers of the project are responsible for running integration tests on PRs before approving them
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah kk sounds good!
README.md
Outdated
* Standard Kubernetes cluster | ||
* Kubernetes Secret | ||
|
||
- Standard Kubernetes cluster (e.g. using [GKE](https://cloud.google.com/kubernetes-engine/) or [Minikube](https://kubernetes.io/docs/setup/minikube/)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure if kaniko works with Minikube actually, maybe we remove this for now?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
kk will remove!
This commit adds docs aimed at folks interested in ramping up and contributing to kaniko. It starts with setting up a github account and forking to make sure the barrier to entry is as low as possible.
6e795fd
to
ac3d364
Compare
Ready for another look @priyawadhwa ! |
LGTM, thank you! |
This commit adds docs aimed at folks interested in ramping up and
contributing to kaniko.
It starts with setting up a github account and forking to make sure the
barrier to entry is as low as possible.