Skip to content

Commit 42cde34

Browse files
Update patlet based on feedback from Russ
1 parent d16c506 commit 42cde34

File tree

1 file changed

+37
-21
lines changed

1 file changed

+37
-21
lines changed

project-roles/trusted-committer.md

Lines changed: 37 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -2,24 +2,35 @@
22

33
Trusted Comitter
44

5-
# Context
5+
# Patlet
66

77
Many inner-source projects will find themselves in a situation where
88
they consistently receive feedback, features, and bug-fixes from a contributor.
99
In these situations project maintainers seek ways to recognize and reward the
1010
work of the contributor above and beyond single contributions.
1111

12+
# Context
13+
14+
- You are the maintainer of a cross-team library, service, or shared resource
15+
- You receive regular contributions
16+
1217
# Problem
1318

14-
A project maintainer receives several contributions from a single contributor
15-
but is unclear on how to properly recognize these contributions.
19+
- Project maintainers want to find ways to scale their ability to support a
20+
project
21+
- Project maintainers want to find ways to scale the longevity of the value
22+
delivered by a project
1623

1724
# Forces
1825

1926
- Over the lifecycle of a project the focus of the maintainers may shift away
2027
to accomodate changing business priorities
21-
- Potential trusted committers seek visible artifacts of their contributions
22-
to community
28+
- Contributors seek visible artifacts of their contributions demonstrating
29+
value
30+
- Motivated contributors looking to push further inner-sourced projects
31+
- Maintaining a project of reasonable complexity is taxing for a small team
32+
- Lack of language for recognizing contributions across teams within an
33+
organization
2334

2435
# Solution
2536

@@ -51,8 +62,8 @@ there is no expectation that candidates will accept the role. Each candidate
5162
should figure out if they have the bandwidth to get involved.
5263

5364
When a candidate accepts the role it is up to the project maintainers to
54-
[publicly recognize the transition from user to TC][praise]. It is also a good
55-
idea to add their name to a Trusted Committers section in your project's
65+
publicly recognize the transition from user to Trusted Committer. It is also a
66+
good idea to add their name to a Trusted Committers section in your project's
5667
README. As an example:
5768

5869
```markdown
@@ -86,25 +97,25 @@ regular basis. A good cadence is every week, but as the Trusted Committer
8697
settles in this can drop to every few weeks or so. The purpose of these
8798
check-ins is to make sure the Trusted Committer feels supported in their new
8899
role, like a 1:1 with your manager. If things aren’t going well, listen and
89-
try to understand what is preventing the TC from being successful. If things
90-
are going well, thank the TC for their continued effort in making the project
91-
successful and set a new date to check-in.
100+
try to understand what is preventing the Trusted Comitter from being successful.
101+
If things are going well, [thank the Trusted Committer for their continued
102+
effort][praise] in making the project successful and set a new date to check-in.
92103

93104
## Sunsetting a Trusted Committer
94105

95106
There comes a time when removal of a Trusted Committer is necessary, such as:
96107

97108
* No longer willing to take part
98109
* No longer able to perform their duties
99-
* No longer employed by Nike
110+
* No longer employed by the company
100111

101112
In each of the above cases a plan for removing access to project resources
102113
should agreed upon by both parties. This includes their entry in a project’s
103114
**Trusted Committer** section.
104115

105-
After access is removed it is courteous to thank the TC for their participation
106-
in a public way. This ensures clear communication and continuity within the
107-
community.
116+
After access is removed it is courteous to [thank the Trusted Committer for
117+
their participation in a public way][praise]. This ensures clear communication
118+
and continuity within the community.
108119

109120
# Resulting Context
110121

@@ -116,30 +127,35 @@ efforts can be used during annual reviews with managers.
116127

117128
## For Maintainers
118129

119-
As a project matures, contributors can become less familiar with key aspects
130+
As a project matures, maintainers can become less familiar with key aspects
120131
of a project. Trusted Committers fill in these gaps. This ensures that all
121132
aspects of the project are better served over time.
122133

123134
A healthy set of Trusted Committers ensures that if project maintainers move on
124-
there is a plan for responsible stewardship at Nike.
135+
there is a plan for responsible stewardship.
125136

126137
# Known Instances
127138

128-
This has been tried and proven successful at Nike.
139+
This has been tried and proven successful at Nike and PayPal.
129140

130141
# Authors
131142

132143
- Fernando Freire
133144

134-
# Acknowledgement
145+
# Acknowledgements
135146

136-
- Russell Rutledge
137-
- Loren Sanz
147+
- [Russell Rutledge]
148+
- [Loren Sanz]
138149
- Noah Cawley
139-
- Jeremy Hicks
150+
- [Jeremy Hicks]
140151

141152
# State
142153

143154
Published internally at Nike; drafted via pull-request in June of 2018.
144155

145156
# Variants
157+
158+
[Russell Rutledge]: https://github.com/rrrutledge
159+
[Loren Sanz]: https://github.com/mrsanz
160+
[Jeremy Hicks]: https://github.com/greatestusername
161+
[praise]: https://github.com/paypal/InnerSourcePatterns/blob/master/praise-participants.md

0 commit comments

Comments
 (0)