pierrejeambrun commented on code in PR #26605:
URL: https://github.com/apache/airflow/pull/26605#discussion_r992524615
##########
dev/stats/get_important_pr_candidates.py:
##########
@@ -267,9 +279,12 @@ def score(self):
# If the body contains over 2000 characters, the PR should matter 40%
more.
# If the body contains fewer than 1000 characters, the PR should
matter 20% less.
#
+ # Weight PRs with #protm tags more heavily:
+ # If there is at least one #protm tag, multiply the interaction score
by the number of tags, up to 3.
+ interaction_score = min(self.interaction_score + 1, 3)
Review Comment:
Maybe I misunderstood, but it looks like we don't take into account the
`self.protm_score`. Should this be:
```
interaction_score *= min(self.protm + 1, 3)
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
