<div dir="auto"><div><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Jan 9, 2024, 8:51 AM Willy Sudiarto Raharjo <<a href="mailto:willysr@slackbuilds.org">willysr@slackbuilds.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">>> I never group PR for unrelated packages.<br>
>> A single commit should only be on one package, not multiple packages.<br>
>><br>
> <br>
> I've always grouped unrelated packages into a single PR. I've asked in the<br>
> past if they'd prefer one PR per package (or related packages) or all in<br>
> one PR and they said it doesn't matter.<br>
> <br>
> I just create an updates branch after each public update and will update my<br>
> packages throughout the week on my local machine. Towards the end of the<br>
> week, I'll push my changes to GitHub and submit a PR.<br>
> <br>
> I have no issues changing my workflow if SBo admins would prefer a certain<br>
> method, but when I've asked in the past, they said whatever works best for<br>
> me.<br>
<br>
basically it doesn't matter whether it's one big PR or separate PR, but <br>
personally i prefer split PR, unless it's a dependency, then it must be <br>
in one big PR<br>
<br>
having split PR is just easier for us (admins) to review the build <br>
process and also to write the trigger command to run the build.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">That works for me. I'll update my process to simplify the admins' job. With how many you guys review, I want to embrace KISS for you guys.</div><div dir="auto"><br></div><div dir="auto">Also, can we trigger the build bot ourselves so it might be finished by the time you see it? I've thought about trying it, but haven't yet.</div><div dir="auto"><br></div><div dir="auto">Jeremy</div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
</blockquote></div></div></div>