Revisions and Commits
| rWIEG wikimedia-iegreview | |||
| rWIEG5dcacdf2e3f8 Add .arcconfig | |||
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | • mmodell | T130418 Phase 1 repository migrations to Differential (goal - end of June 2016) | |||
| Resolved | • mmodell | T132174 Migrate wikimedia-iegreview to Differential |
Event Timeline
My proposals:
- 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)
@Luke081515 Thanks :) But that's actually an answer to a different question :)
I am more interested in things like CI needs. My mistake for being too quick and not being specific.
Ok I got harbormaster CI set up for rWIEG wikimedia-iegreview: the most recent commit ran composer test successfully, see {B604}
All that remains is to set the gerrit repo to read-only and start using differential for code reviews.
rWWSC Wikimania Scholarships also builds with harbormaster: https://integration.wikimedia.org/ci/job/phab-ci-jessie/68/console
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.