Update README.md
This commit is contained in:
parent
07404db9a2
commit
75f28322cc
1 changed files with 1 additions and 28 deletions
29
README.md
29
README.md
|
@ -5,34 +5,6 @@ This repository contains the configuration files for the pre-merge checks for th
|
|||
# User documentation
|
||||
See [docs/user_doc.md](docs/user_doc.md)
|
||||
|
||||
# Pre-merge check vision for end of 2019
|
||||
Maria is a developer working on a new idea for the LLVM project. When she submits a new diff to Phabricator (or updates an existing diff), the pre-merge checks are triggered automatically in the cloud. The pre-merge checks run in one configuration (amd64, Debian Testing, clang8) and comprise these steps:
|
||||
|
||||
* run CMake
|
||||
* compile and link
|
||||
* run check-all
|
||||
* run clang-format and clang-tidy
|
||||
|
||||
Once a check is completed, the results are attached to the Diff in Phabricator so that Maria is notified automatically with the results. Now she can check if the pre-merge checks passed and fix things as required. This way she does not have to run all checks on her local machine by hand. The first results are available within 2 hours of a change, but typically within 30 minutes.
|
||||
|
||||
As the pre-merge check cover the easy parts, a human reviewer can focus on the parts that the machine cannot cover. The pre-merge checks are optional, so a reviewer can also decide to ignore them if they do not work, take too long, or do not make sense.
|
||||
The build bots are still in place as they cover a much wider range of checks on the different platforms. So after Maria’s change has landed, the build bots might still find more bugs that were not found in the pre-merge checks. The number should be lower than what we have today though.
|
||||
|
||||
# Roadmap
|
||||
|
||||
During the LLVM developer meeting in October 2019, this was the roadmap we discussed:
|
||||
* Until mid of November: preparation of the infrastructure for the beta testing
|
||||
* Mid of November:
|
||||
* Announcement of the public beta testing phase on the LLVM mailing list.
|
||||
* Interested users are white-listed on the Herald rule.
|
||||
* Until January: Beta testing for interested users.
|
||||
* In January:
|
||||
* Based on the feedback so far: enable pre-merge tests for all users.
|
||||
* Users can opt-out via a black-list in the Herald rule.
|
||||
* Start discussion with LLVM foundation on a permanent setup/maintenance for the pre-merge tests.
|
||||
|
||||
This roadmap is also reflected in the [Milestones](https://github.com/google/llvm-premerge-checks/milestones?direction=asc&sort=due_date&state=open).
|
||||
|
||||
# Cluster overview
|
||||
|
||||
The cluster consists of these services:
|
||||
|
@ -61,6 +33,7 @@ There is no backup of the credentials. If you need to change it, generate a new
|
|||
# Additional Information
|
||||
* [Playbooks](docs/playbooks.md) for installing/upgrading
|
||||
* [User documentation](docs/user_doc.md)
|
||||
* [Log of the service operations](https://github.com/google/llvm-premerge-checks/wiki/LLVM-pre-merge-tests-operations-blog)
|
||||
|
||||
# License
|
||||
This project is licensed unter the "Apache 2.0 with LLVM Exception" license. See [LICENSE](LICENSE) for details.
|
||||
|
|
Loading…
Reference in a new issue