Anonymitaet commented on code in PR #111: URL: https://github.com/apache/pulsar-site/pull/111#discussion_r896440054
########## site2/website-next/src/pages/contributing.md: ########## @@ -156,6 +156,97 @@ Then you can push your local, committed changes to your (forked) repository on G All code should have appropriate unit testing coverage. New code should have new tests in the same contribution. Bug fixes should include a regression test to prevent the issue from reoccurring. +### CI Testing in your Fork + +The following is copied from the +[thread](https://lists.apache.org/thread/1ofdbd9j1hhj4c0rw3o7vv8y6rbsxrp0) and [#15385](https://github.com/apache/pulsar/pull/15385). + + +#### Each PR triggers many CI Jobs +Since we cannot add more resources to Pulsar CI, we need to find other ways +to reduce the load on Pulsar CI. +Every developer can help reduce the load on Pulsar CI. +Please don't open a PR to apache/pulsar when the work is in very initial +stages. +Every push to the pull request branch will trigger a lot of concurrent +builds (about 25). +By reducing the amount of PR pushes you can reduce the load on Pulsar CI. + +#### Run CI from Fork +You don't have to stop developing and using CI for getting feedback. There +is a better way to get CI feedback for early stages of development. The +secret is that everyone has their personal CI on GitHub! Review Comment: @maxsxu FYI ########## site2/website-next/src/pages/contributing.md: ########## @@ -156,6 +156,97 @@ Then you can push your local, committed changes to your (forked) repository on G All code should have appropriate unit testing coverage. New code should have new tests in the same contribution. Bug fixes should include a regression test to prevent the issue from reoccurring. +### CI Testing in your Fork + +The following is copied from the +[thread](https://lists.apache.org/thread/1ofdbd9j1hhj4c0rw3o7vv8y6rbsxrp0) and [#15385](https://github.com/apache/pulsar/pull/15385). + + +#### Each PR triggers many CI Jobs +Since we cannot add more resources to Pulsar CI, we need to find other ways Review Comment: @maxsxu FYI -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@pulsar.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
