ray-project / kuberay

A toolkit to run Ray applications on Kubernetes
Apache License 2.0
1.13k stars 366 forks source link

[Feature] Pre-push hooks #559

Open DmitriGekhtman opened 2 years ago

DmitriGekhtman commented 2 years ago

Search before asking

Description

It would be great to have a pre-push hook for the KubeRay repo. Every lightweight check in the CI should be included as part of the hook. Things to include:

If the hook modifies code, it should fail.

Guidance on install the hook should be included in the Development README.

Use case

It is convenient for development to have your push blocked by a pre-push failure. Otherwise, you have to go to the GitHub UI and wait for checks to run. This slows development iteration speed.

Related issues

No response

Are you willing to submit a PR?

DmitriGekhtman commented 2 years ago

Scripts to be run at the discretion of the developer are an alternative.