You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Copy file name to clipboardExpand all lines: patterns/1-initial/capacity-for-contributions.md
+9-9Lines changed: 9 additions & 9 deletions
Display the source diff
Display the rich diff
Original file line number
Diff line number
Diff line change
@@ -1,24 +1,24 @@
1
1
## Title
2
2
3
-
Mananging capacity for reviewing contributions
3
+
Managing capacity for reviewing contributions
4
4
5
5
## Patlet
6
6
7
7
Reviewing InnerSource contributions takes time and effort. This should be reflected in capacity planning, especially for larger contributions. Expectations and available capacity should be transparent so that contributors understand when their contributions will be reviewed and released.
8
8
9
9
## Problem
10
10
11
-
Large InnerSource contributions are causing delays to other work in the team and/or contributons are taking longer to be released than expected. Reviewing contributions may be significant invisible work, not tracked in a team's agile development process.
11
+
Large InnerSource contributions are causing delays to other work in the team and/or contributions are taking longer to be released than expected. Reviewing contributions may be significant invisible work, not tracked in a team's agile development process.
12
12
13
13
## Story
14
14
15
15
The BBC's connected TV application are built by a number of teams, each with different areas of responsibility. They work on each other's areas of the codebase via InnerSource on a regular basis.
16
16
17
-
The team reapsonsible for the build process for the JavaScript bundles received a major pull request, changing how depedencies were bundled. This PR introduced a new build time dependency, a new strucutre to the deployed JavaSctipt bundles, and touched 503 files, with 6,699 lines of code added and 2,828 lines of code deleted. A contribution of this size required signicant time to code review, test, and ensure the team understood the new tooling snd structure introduced. Normally, InnerSource contriobutions would be reviewed ad-hoc, but the team estimated that this review process would take days rather than hours. Reviewing this PR would have delayed the team's other work, so the team raised this with the team lead, delivery manager, and product manager, to prioritise this work against other work. Time was set aside to review this contribution at future date.
17
+
The team responsible for the build process for the JavaScript bundles received a major pull request, changing how dependencies were bundled. This PR introduced a new build time dependency, a new structure to the deployed JavaScript bundles, and touched 503 files, with 6,699 lines of code added and 2,828 lines of code deleted. A contribution of this size required significant time to code review, test, and ensure the team understood the new tooling and structure introduced. Normally, InnerSource contributions would be reviewed ad-hoc, but the team estimated that this review process would take days rather than hours. Reviewing this PR would have delayed the team's other work, so the team raised this with the team lead, delivery manager, and product manager, to prioritize this work against other work. Time was set aside to review this contribution at future date.
18
18
19
-
This process was formalised in the team:
19
+
This process was formalized in the team:
20
20
21
-
* Larger contributions have a ticket created on the team's backlog and included in prioritisation disacussion alongside other tickets. The contributor will be informed of the priority call and given an estimate as to when it will be reviewed and released.
21
+
* Larger contributions have a ticket created on the team's backlog and included in prioritization discussion alongside other tickets. The contributor will be informed of the priority call and given an estimate as to when it will be reviewed and released.
22
22
* Smaller contributions can still be reviewed ad-hoc.
23
23
24
24
## Context
@@ -30,15 +30,15 @@ Maintainers of a successful InnerSource project are finding it difficult to revi
30
30
* Contributors expect timely feedback on their contributions
31
31
* Maintaining team are expected to deliver other work alongside reviewing contributions
32
32
* Missing communication between contributors and maintainers on expectations/lead time for contributions to be reviewed/released
33
-
* Tension in prioritising InnerSource contributions against other work
33
+
* Tension in prioritizing InnerSource contributions against other work
34
34
35
35
## Solutions
36
36
37
37
* Reviewing larger contributions is tracked in the team's ticketing system/bug tracker (e.g. Jira, GitHub issues)
38
-
*Maintaing team is given time to review larger contributions in team capacity planning
39
-
* Reviewing contributions is prioritised against other work (e.g. in sprint planning)
38
+
*Maintaining team is given time to review larger contributions in team capacity planning
39
+
* Reviewing contributions is prioritized against other work (e.g. in sprint planning)
40
40
* Maintainers communicate their capacity for reviewing contributions, the priority of contributions, and an estimate of when a contribution will be reviewed/released
41
-
*Maintaing team has a service level objective (SLO) (e.g. 2 working days) for contributions receiving initial feedback
41
+
*Maintaining team has a service level objective (SLO) (e.g. 2 working days) for contributions receiving initial feedback
0 commit comments