- Follow the design document policy.
- File a GitHub issue.
- Implement the change.
- Update labels.
- Update repositories.
- Flip the incompatible flag.
GitHub issue
File a GitHub issue in the Bazel repository. See example. We recommend that:-
The title starts with the name of the flag (the flag name will start with
incompatible_). -
You add the label
incompatible-change. - The description contains a description of the change and a link to relevant design documents.
- The description contains a migration recipe, to explain users how they should update their code. Ideally, when the change is mechanical, include a link to a migration tool.
- The description includes an example of the error message users will get if they don’t migrate. This will make the GitHub issue more discoverable from search engines. Make sure that the error message is helpful and actionable. When possible, the error message should include the name of the incompatible flag.
BUILD, WORKSPACE, and .bzl files.
It may also report warnings.
Implementation
Create a new flag in Bazel. The default value must be false. The help text should contain the URL of the GitHub issue. As the flag name starts withincompatible_, it needs metadata tags:
RELNOTES: in the following form:
RELNOTES: --incompatible_name_of_flag has been added. See #xyz for details
The commit should also update the relevant documentation, so that there is no
window of commits in which the code is inconsistent with the docs. Since our
documentation is versioned, changes to the docs will not be inadvertently
released prematurely.
Labels
Once the commit is merged and the incompatible change is ready to be adopted, add the labelmigration-ready
to the GitHub issue.
If a problem is found with the flag and users are not expected to migrate yet:
remove the flags migration-ready.
If you plan to flip the flag in the next major release, add label `breaking-change-X.0” to the issue.
Updating repositories
Bazel CI tests a list of important projects at Bazel@HEAD + Downstream. Most of them are often dependencies of other Bazel projects, therefore it’s important to migrate them to unblock the migration for the broader community. To monitor the migration status of those projects, you can use thebazelisk-plus-incompatible-flags pipeline.
Check how this pipeline works here.
Our dev support team monitors the migration-ready label. Once you add this label to the GitHub issue, they will handle the following:
- Create a comment in the GitHub issue to track the list of failures and downstream projects that need to be migrated (see example)
- File Github issues to notify the owners of every downstream project broken by your incompatible change (see example)
- Follow up to make sure all issues are addressed before the target release date
- Send PRs to fix downstream projects.
- Reach out to the Bazel community for help on migration (e.g. Bazel Rules Authors SIG).
Flipping the flag
Before flipping the default value of the flag to true, please make sure that:-
Core repositories in the ecosystem are migrated.
On the
bazelisk-plus-incompatible-flagspipeline, the flag should appear underThe following flags didn't break any passing Bazel team owned/co-owned projects. - All issues in the checklist are marked as fixed/closed.
- User concerns and questions have been resolved.
blazerc file to unblock the flag flip. By doing this, we can ensure Bazel users depend on the new behaviour by default as early as possible.
When changing the flag default to true, please:
- Use
RELNOTES[INC]in the commit description, with the following format:RELNOTES[INC]: --incompatible_name_of_flag is flipped to true. See #xyz for detailsYou can include additional information in the rest of the commit description. - Use
Fixes #xyzin the description, so that the GitHub issue gets closed when the commit is merged. - Review and update documentation if needed.
- File a new issue
#abcto track the removal of the flag.
Removing the flag
After the flag is flipped at HEAD, it should be removed from Bazel eventually. When you plan to remove the incompatible flag:- Consider leaving more time for users to migrate if it’s a major incompatible change. Ideally, the flag should be available in at least one major release.
- For the commit that removes the flag, use
Fixes #abcin the description so that the GitHub issue gets closed when the commit is merged.