Old PRs needing reviews

[antoviaque]

Cf https://discuss.openedx.org/t/core-contributor-news-for-sprint-mar-29th-apr-11th-2025/15552/15

The[ list of stuck PR](The list of stuck PR shows some pretty old PR - the oldest two being from 2023 (2nd PR), and don’t look too big or controversial? Are there for those to have been there for so long? What could we do to unblock them?) shows some pretty old PR - the oldest two being from 2023 (2nd PR), and don’t look too big or controversial? Why have they been there for so long? What could we do to unblock them?

[antoviaque]

Many of these PRs have been updated/pushed forward over the past few weeks - we can come back to this in a little while to see if that helped.

[cassiezamparini]

@antoviaque This is awesome news! Do you think mentioning them in the "CC Newsletter" is one thing that's helping?

[antoviaque]

@cassiezamparini Definitely, it made some of us go have a look yes - and especially @sarina went to help moving them (thank you!).

[cassiezamparini]

What a win! Thanks for the update @antoviaque

[sarina]

Yep I try to take a look every sprint - it's helpful!

[sarina]

It might be worth refining it a bit, some of the oldest PRs now have comments explaining why they're stuck and I'm not expecting to see movement on them.

[antoviaque]

@sarina Good point - maybe we could have a narrow label to exclude specific tickets from being listed; maybe "no-cc-help-needed-now" or something? Maybe @itsjeyd will have a better idea :)

[sarina]

Or exclude ones that are in the "Waiting on Author" state?

[itsjeyd]

@antoviaque @sarina I forget how the list of stuck PRs in the CC newsletter is being compiled -- if it's based solely on how long ago they were opened, it would definitely make sense to add some filtering based on the waiting on author state and/or label.

CC @cassiezamparini

[sarina]

Hi, I'm curious if we're only putting old edx-platform PRs in the newsletter, rather than all oldest PRs across the org?

[itsjeyd]

[inform] Conversation continues here.

[sarina]

I added the comment in the forums because no one responded to me here. I think we need to update the sprint report:

• 10 oldest PRs across the project
• Excluding PRs that are in the "Waiting on Author" state.