Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

XS✔ ◾ Merge debt - add video #9433

Merged
merged 2 commits into from
Oct 17, 2024
Merged

XS✔ ◾ Merge debt - add video #9433

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
a9cc4cb
Merge debt - add video
bradystroud Oct 17, 2024
eea4830
Fix newlines
bradystroud Oct 17, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 9 additions & 2 deletions rules/merge-debt/rule.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,10 @@ authors:
url: https://www.ssw.com.au/people/brook-jeynes
- title: Gordon Beeming
url: https://ssw.com.au/people/gordon-beeming
- title: Matt Goldman
url: https://ssw.com.au/people/matt-goldman
- title: Luke Cook
url: https://ssw.com.au/people/luke-cook
- title: Adam Cogan
url: https://ssw.com.au/people/adam-cogan
related:
Expand All @@ -22,10 +26,13 @@ When a PR is created but left open for a long time, it becomes stale. Stale PRs

<!--endintro-->

If you are in a team (i.e. an internal contributor), it is the PR author's responsibility to get a PR reviewed and action feedback ASAP.
`youtube: https://www.youtube.com/watch?v=SJdq7mLGdgA`
**Video: Merge Debt | Matt Goldman & Luke Cook | Rules (16 min)**

If you are in a team (i.e. an internal contributor), it is the PR author's responsibility to get a PR reviewed and action feedback ASAP.
**Tip:** For internal contributors, it's a good idea to have a [call with the PR reviewers](/over-the-shoulder).

If you are **not** part of the team (i.e. an outside contributor), reviewing the PR is the responsibility of the Repo maintainers. Actioning the feedback is still the responsibility of the PR author.
If you are **not** part of the team (i.e. an outside contributor), reviewing the PR is the responsibility of the Repo maintainers. Actioning the feedback is still the responsibility of the PR author.
**Tip:** For outside contributors, it's a good idea to chase the reviewers by reaching out with a comment on GitHub, or through the repo's community (e.g. Discord channels).

::: info
Expand Down
Loading