Skip to content
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

[Doc] Run pre-commit on cluster docs #47342

Merged

Conversation

peytondmurray
Copy link
Contributor

Why are these changes needed?

Currently we have no linting on any part of the docs code. This PR runs pre-commit on the cluster docs.

This PR fixes the following issues:

trim trailing whitespace.................................................Failed
- hook id: trailing-whitespace
- exit code: 1
- files were modified by this hook

Fixing doc/source/cluster/kubernetes/user-guides/aws-eks-gpu-cluster.md
Fixing doc/source/cluster/running-applications/job-submission/cli.rst
Fixing doc/source/cluster/configure-manage-dashboard.md
Fixing doc/source/cluster/kubernetes/user-guides/pod-security.md
Fixing doc/source/cluster/vms/user-guides/launching-clusters/vsphere.md
Fixing doc/source/cluster/kubernetes/user-guides/helm-chart-rbac.md
Fixing doc/source/cluster/vms/references/ray-cluster-configuration.rst
Fixing doc/source/cluster/running-applications/job-submission/quickstart.rst
Fixing doc/source/cluster/kubernetes/examples/stable-diffusion-rayservice.md
Fixing doc/source/cluster/kubernetes/getting-started/raycluster-quick-start.md
Fixing doc/source/cluster/kubernetes/examples/rayjob-kueue-gang-scheduling.md
Fixing doc/source/cluster/kubernetes/k8s-ecosystem/ingress.md
Fixing doc/source/cluster/kubernetes/user-guides/kuberay-gcs-ft.md
Fixing doc/source/cluster/kubernetes/configs/static-ray-cluster-networkpolicy.yaml
Fixing doc/source/cluster/kubernetes/k8s-ecosystem/pyspy.md
Fixing doc/source/cluster/kubernetes/k8s-ecosystem/volcano.md
Fixing doc/source/cluster/running-applications/job-submission/sdk.rst
Fixing doc/source/cluster/running-applications/job-submission/ray-client.rst
Fixing doc/source/cluster/kubernetes/troubleshooting/troubleshooting.md
Fixing doc/source/cluster/kubernetes/getting-started/rayjob-quick-start.md
Fixing doc/source/cluster/kubernetes/configs/ray-cluster.gpu.yaml
Fixing doc/source/cluster/kubernetes/configs/static-ray-cluster.with-fault-tolerance.yaml
Fixing doc/source/cluster/kubernetes/examples/mnist-training-example.md
Fixing doc/source/cluster/kubernetes/configs/static-ray-cluster.tls.yaml
Fixing doc/source/cluster/kubernetes/user-guides/gcp-gke-gpu-cluster.md
Fixing doc/source/cluster/kubernetes/examples/distributed-checkpointing-with-gcsfuse.md
Fixing doc/source/cluster/kubernetes/user-guides/gke-gcs-bucket.md
Fixing doc/source/cluster/kubernetes/user-guides/logging.md
Fixing doc/source/cluster/kubernetes/examples/text-summarizer-rayservice.md
Fixing doc/source/cluster/kubernetes/examples/rayjob-batch-inference-example.md
Fixing doc/source/cluster/metrics.md
Fixing doc/source/cluster/kubernetes/k8s-ecosystem/kubeflow.md
Fixing doc/source/cluster/kubernetes/k8s-ecosystem/kueue.md
Fixing doc/source/cluster/kubernetes/examples/rayjob-kueue-priority-scheduling.md
Fixing doc/source/cluster/faq.rst
Fixing doc/source/cluster/running-applications/job-submission/openapi.yml
Fixing doc/source/cluster/kubernetes/user-guides/configuring-autoscaling.md
Fixing doc/source/cluster/kubernetes/getting-started/rayservice-quick-start.md
Fixing doc/source/cluster/kubernetes/user-guides/static-ray-cluster-without-kuberay.md
Fixing doc/source/cluster/kubernetes/user-guides/config.md
Fixing doc/source/cluster/kubernetes/user-guides/pod-command.md

fix end of files.........................................................Failed
- hook id: end-of-file-fixer
- exit code: 1
- files were modified by this hook

Fixing doc/source/cluster/kubernetes/images/rbac-clusterrole.svg
Fixing doc/source/cluster/running-applications/job-submission/cli.rst
Fixing doc/source/cluster/vms/user-guides/community/slurm.rst
Fixing doc/source/cluster/kubernetes/benchmarks/memory-scalability-benchmark.md
Fixing doc/source/cluster/images/ray-job-diagram.svg
Fixing doc/source/cluster/kubernetes/user-guides/observability.md
Fixing doc/source/cluster/kubernetes/examples/stable-diffusion-rayservice.md
Fixing doc/source/cluster/kubernetes/configs/static-ray-cluster-networkpolicy.yaml
Fixing doc/source/cluster/kubernetes/images/rbac-role-one-namespace.svg
Fixing doc/source/cluster/kubernetes/examples/mnist-training-example.md
Fixing doc/source/cluster/cli.rst
Fixing doc/source/cluster/kubernetes/configs/static-ray-cluster.tls.yaml
Fixing doc/source/cluster/kubernetes/user-guides/gcp-gke-gpu-cluster.md
Fixing doc/source/cluster/kubernetes/user-guides/logging.md
Fixing doc/source/cluster/kubernetes/examples/text-summarizer-rayservice.md
Fixing doc/source/cluster/kubernetes/images/rbac-role-multi-namespaces.svg
Fixing doc/source/cluster/kubernetes/images/kubeflow-architecture.svg
Fixing doc/source/cluster/faq.rst
Fixing doc/source/cluster/running-applications/job-submission/openapi.yml
Fixing doc/source/cluster/kubernetes/images/AutoscalerOperator.svg

check for added large files..............................................Passed
check python ast.........................................................Passed
check json...........................................(no files to check)Skipped
check toml...........................................(no files to check)Skipped
black....................................................................Passed
flake8...................................................................Passed
prettier.............................................(no files to check)Skipped
mypy.................................................(no files to check)Skipped
isort (python)...........................................................Passed
rst directives end with two colons.......................................Passed
rst ``inline code`` next to normal text..................................Passed
use logger.warning(......................................................Passed
check for not-real mock methods..........................................Passed
ShellCheck v0.9.0........................................................Passed
clang-format.........................................(no files to check)Skipped
Google Java Formatter................................(no files to check)Skipped
Check for Ray docstyle violations........................................Passed
Check for Ray import order violations....................................Passed

Related issue number

Part of #42587.

Checks

  • I've signed off every commit(by using the -s flag, i.e., git commit -s) in this PR.
  • I've run scripts/format.sh to lint the changes in this PR.
  • I've included any doc changes needed for https://docs.ray.io/en/master/.
    • I've added any new APIs to the API Reference. For example, if I added a
      method in Tune, I've added it in doc/source/tune/api/ under the
      corresponding .rst file.
  • I've made sure the tests are passing. Note that there might be a few flaky tests, see the recent failures at https://flakey-tests.ray.io/
  • Testing Strategy
    • Unit tests
    • Release tests
    • This PR is not tested :(

Signed-off-by: pdmurray <peynmurray@gmail.com>
@peytondmurray peytondmurray added docs An issue or change related to documentation clusters labels Aug 26, 2024
@can-anyscale can-anyscale enabled auto-merge (squash) August 29, 2024 19:41
@github-actions github-actions bot added the go add ONLY when ready to merge, run all tests label Aug 29, 2024
@can-anyscale can-anyscale merged commit 7cdf126 into ray-project:master Aug 29, 2024
6 checks passed
@peytondmurray peytondmurray deleted the 42587-pre-commit-docs-cluster branch August 29, 2024 20:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
clusters docs An issue or change related to documentation go add ONLY when ready to merge, run all tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants