mirror of
https://github.com/ceph/ceph-csi.git
synced 2024-12-18 02:50:30 +00:00
d5a0606c33
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
1.2 KiB
1.2 KiB
Contributing to Ginkgo
Your contributions to Ginkgo are essential for its long-term maintenance and improvement.
- Please open an issue first - describe what problem you are trying to solve and give the community a forum for input and feedback ahead of investing time in writing code!
- Ensure adequate test coverage:
- When adding to the Ginkgo library, add unit and/or integration tests (under the
integration
folder). - When adding to the Ginkgo CLI, note that there are very few unit tests. Please add an integration test.
- When adding to the Ginkgo library, add unit and/or integration tests (under the
- Update the documentation. Ginko uses
godoc
comments and documentation on thegh-pages
branch. If relevant, please submit a docs PR to that branch alongside your code PR.
Thanks for supporting Ginkgo!
Setup
Fork the repo, then:
go get github.com/onsi/ginkgo
go get github.com/onsi/gomega/...
cd $GOPATH/src/github.com/onsi/ginkgo
git remote add fork git@github.com:<NAME>/ginkgo.git
ginkgo -r -p # ensure tests are green
go vet ./... # ensure linter is happy
Making the PR
- go to a new branch
git checkout -b my-feature
- make your changes
- run tests and linter again (see above)
git push fork
- open PR 🎉