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: CONTRIBUTING.md
+1-5Lines changed: 1 addition & 5 deletions
Display the source diff
Display the rich diff
Original file line number
Diff line number
Diff line change
@@ -25,11 +25,7 @@ Thank you for contributing!
25
25
26
26
## PR reviews
27
27
28
-
For feedback in PRs, we use the [LOGAF scale](https://blog.danlew.net/2020/04/15/the-logaf-scale/) to specify how important a comment is:
29
-
30
-
*`l`: low - nitpick. You may address this comment, but you don't have to.
31
-
*`m`: medium - normal comment. Worth addressing and fixing.
32
-
*`h`: high - Very important. We must not merge this PR without addressing this issue.
28
+
For feedback in PRs, we use the [LOGAF scale](https://develop.sentry.dev/engineering-practices/code-review/#logaf-scale) to specify how important a comment is.
33
29
34
30
You only need one approval from a maintainer to be able to merge. For some PRs, asking specific or multiple people for review might be adequate.
0 commit comments