bazelbuild / bazelbuild/rules_rust

Document pre-submit procedure.

Open
#2,037 0 comments 0 reactions 0 assignees View on GitHub
needs-triage
Dominant language
Starlark
Stars
843
Forks
651
Avg merge
2d 18h
Merged PRs (30d)
15

Description

As a new contributor, it is not clear what steps should be done to test a change locally before sending a PR. Some ideas of information to include based on issues I've encountered:

- Which tests to run and the respective `bazel` commands to run them.
- Info on updating docs (CI will let you know about `./docs/update_docs.sh` but relying on that adds latency to the PR process)
- Running `buildifier`. (It's unclear from a failed CI run how to invoke `buildifier` with the same options that are used in CI. CI does not print out the command here. Failures here also add latency to the PR process).

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.