|Resolved||• mmodell||T130418 Phase 1 repository migrations to Differential (goal - end of June 2016)|
|Resolved||• mmodell||T132174 Migrate wikimedia-iegreview to Differential|
- Create an owners package for that repo
- Add the folks to that package, who should at least review changes (one of them at least for a change)
- Create an herald rule, which blocks commits, which are not reviewed (you can exception per branch, or per commiter)
- Adjust push policy: Who should be able to push? If you don't want everyone to push changes, maybe only people of the packages maybe a good soluation.
remember: If you require at least one member of a package to review the patch, the package get's not added, if the author of the revision is an owner of that package (see https://secure.phabricator.com/T10939#176633), and: The package can get removed, so people with bad faith can commit actually anyway :-/ (if push policy is not adjusted, see above)
If you want, I'm happy to setup these condidtions, I just need the relevant data. (Owners etc)
Ok I got harbormaster CI set up for rWIEG wikimedia-iegreview: the most recent commit ran composer test successfully, see B604: rWIEGefa1a2c076ed: Add php_codesniffer to `composer test`
All that remains is to set the gerrit repo to read-only and start using differential for code reviews.
rWIEG wikimedia-iegreview is now hosted in diffusion, mirrored to gerrit and the gerrit repository is locked. This way you can keep the old remote URI and deploy with trebuchet until this gets moved to scap3. Commits must be pushed to phabricator and reviews should go through differential. I added a .arcconfig in the repository so that arcanist will know what to do.