diff options
author | aiuto <aiuto@google.com> | 2022-11-04 15:56:21 -0400 |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-11-04 15:56:21 -0400 |
commit | bcc76c8818e70ef3990d506bc3b968c34ab14cfb (patch) | |
tree | 2de6cbc1be56e86922e6d95af180904602237fce | |
parent | d7b20dacb0e9ee10e5e94d914ff60821fdd8a63c (diff) | |
download | bazelbuild-rules_pkg-bcc76c8818e70ef3990d506bc3b968c34ab14cfb.tar.gz |
Update patching.md
Tidying
-rw-r--r-- | patching.md | 33 |
1 files changed, 23 insertions, 10 deletions
diff --git a/patching.md b/patching.md index de7c459..b537ffc 100644 --- a/patching.md +++ b/patching.md @@ -1,30 +1,43 @@ # Patch Acceptance Process -1. PRs that change or add behavior are not accepted without being tied to an +- PRs that change or add behavior are not accepted without being tied to an issue. Most fixes, even if you think they are obvious, require an issue too. Almost every change breaks someone who has depended on the behavior, even broken behavior. -1. Significant changes need a design document. Please create an issue describing +- Significant changes need a design document. Please create an issue describing the proposed change, and post a link to it to rules-pkg-discuss@googlegroups.com. Wait for discussion to come to agreement before proceeding. -1. Prepare a git commit that implements the feature. Don't forget to add tests. -1. Ensure you've signed a [Contributor License +- Features and bug fixes should be as portable as possible. + - do not not disable tests on Windows because it is convenient for you + - if a feature is only available on specific platforms, it must be optional. That + is, it requires a distinct bzlmod MODULE +- All fixes and features must have tests. +- Ensure you've signed a [Contributor License Agreement](https://cla.developers.google.com). -1. Send us a pull request on +- Send us a pull request on [GitHub](https://github.com/bazelbuild/rules_pkg/pulls). If you're new to GitHub, read [about pull requests](https://help.github.com/articles/about-pull-requests/). Note that we restrict permissions to create branches on the main repository, so you will need to push your commit to [your own fork of the repository](https://help.github.com/articles/working-with-forks/). -1. Wait for a repository owner to assign you a reviewer. We strive to do that +- Wait for a repository owner to assign you a reviewer. We strive to do that within 4 business days, but it may take longer. If your review gets lost you can escalate by sending a mail to [bazel-dev@googlegroups.com](mailto:bazel-dev@googlegroups.com). -1. Work with the reviewer to complete a code review. For each change, create a +- Work with the reviewer to complete a code review. For each change, create a new commit and push it to make changes to your pull request. -1. A maintainer will approve the PR and merge it. +- A maintainer will approve the PR and merge it. + +Tips +- Large PRs are harder to review. If you have to refactor code to implement a feature + please split that into at least 2 PRs. The first to refactor without changing behavior + and the second to implemtn the new behavior. Of course, as above, any PR that large + should be discussed in an issue first +- Please do not send PRs that update dependencies (WORKSPACE or MODULE.bzl) just to + stay at head. We try to maintain backwards compatibility to LTS releases as long as + possible, so we only update to new versions of dependencies when it is required. For further information about working with Bazel and rules in general: -1. Read the [Bazel governance plan](https://www.bazel.build/governance.html). -1. Read the [contributing to Bazel](https://www.bazel.build/contributing.html) guide. +- Read the [Bazel governance plan](https://www.bazel.build/governance.html). +- Read the [contributing to Bazel](https://www.bazel.build/contributing.html) guide. |