This message was deleted.
# report-bugss
Slackbot
06/23/2023, 4:52 PMThis message was deleted.
x
Xiulung Choy
06/23/2023, 4:53 PMHey Ricardo, Graphite takes a look at the code owner requirements from the branch you will eventually be merging into. Which will differ from what GitHub shows if you’re looking at an upstack PR
r
Ricardo Delfin
06/23/2023, 4:53 PMHmmm... This one should be merging straight into main as far as I can tell
Ricardo Delfin
06/23/2023, 4:54 PMI do also have a bot commenting the min set of reviewers needed for a PR (code here) and it seems to agree that the PR is good to merge
b
Brendan Ngo
06/23/2023, 5:07 PMHey sorry about this! I'll take a look
r
Ricardo Delfin
06/23/2023, 5:08 PMNo worries, thanks!
We do have really weird owners 😅
r
Ricardo Delfin
06/27/2023, 2:21 PMHi! I think this issue is back/still there @Brendan Ngo: https://app.graphite.dev/github/pr/wayveai/WayveCode/34169
I see this at the top:
b
Brendan Ngo
06/27/2023, 2:42 PMOop let me take a look!
Brendan Ngo
06/27/2023, 3:01 PMalso can you leave the pr as is for a little bit
Brendan Ngo
06/27/2023, 3:01 PMdebugging right now!
r
Ricardo Delfin
06/27/2023, 3:01 PMHahah yeah sorry, I keep on rebasing
Ricardo Delfin
06/27/2023, 3:01 PMI'll leave it as is and hope CI doesn't push a commit to it
Ricardo Delfin
06/27/2023, 3:02 PMThanks!