Complete the peer review section on actually reviewing differentials, and landing them.
ClosedPublic
Actions

Authored by vanmeeuwen on Mar 14 2016, 3:47 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sat, Oct 12, 9:31 PM

	Unknown Object (File)
	Fri, Oct 4, 8:16 PM

	Unknown Object (File)
	Fri, Oct 4, 8:13 PM

	Unknown Object (File)
	Fri, Oct 4, 8:12 PM

	Unknown Object (File)
	Fri, Oct 4, 8:10 PM

	Unknown Object (File)
	Tue, Sep 24, 3:29 PM

	Unknown Object (File)
	Sep 11 2024, 8:42 AM

	Unknown Object (File)
	Sep 7 2024, 6:19 AM

View All Files

Subscribers

petersen

Tokens

"Like" token, awarded by petersen.

Details

Reviewers

petersen

Group Reviewers

Documentation Authors

Maniphest Tasks

T1095: Mandatory Peer Review Process

Commits

rDd5fbb58d877d: Complete the peer review section on actually reviewing differentials, and…

Summary

Resolves T1095

Test Plan

Read through it.

Diff Detail

Repository

rD docs

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

vanmeeuwen updated this revision to Diff 214.Mar 14 2016, 3:47 PM

vanmeeuwen retitled this revision from to Complete the peer review section on actually reviewing differentials, and landing them..

vanmeeuwen updated this object.

vanmeeuwen edited the test plan for this revision. (Show Details)

vanmeeuwen added a reviewer: Documentation Authors.

vanmeeuwen added a task: T1095: Mandatory Peer Review Process.

Love all the congratulations. we need more of that.. :-)

This revision is now accepted and ready to land.Mar 24 2016, 12:17 PM

It's a go from my perspective. I don't seem to be able to approve it?

Closed by commit rDd5fbb58d877d: Complete the peer review section on actually reviewing differentials, and… (authored by vanmeeuwen, committed by vanmeeuwen). · Explain WhyApr 4 2016, 9:45 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

source/

contributor-guide/

peer-review.rst

56 lines

Diff 244

View Options

source/contributor-guide/peer-review.rst

Complete the peer review section on actually reviewing differentials, and landing them.ClosedPublicActions