sPHENIX discussion of software

[pinkenburg <pinkenburg AT bnl.gov>]

Hi folks,

one more thing - the "sync fork" button merges your PR with the master 
which will trigger jenkins again. I think this is the reason why I see 
so many of those jenkins runs. Just don't use this.

Thanks
Chris


On 6/14/2024 8:27 AM, pinkenburg via sPHENIX-software-l wrote:
> Hi folks,
>
> jenkins has a hard time keeping up with the number of PRs which are 
> submitted. No problem with that but we have to make sure that this 
> does not become a bottleneck for our development. Here are a few tips 
> to alleviate the load:
>
> This is the biggest one: Every push into a PR triggers a fresh jenkins 
> run. I have seen users pushing changes into their PR after every 
> commit on their local repo many times within minutes: Please commit 
> everything first and then do a single push
>
> Make sure your code (or PR change) actually compiles.
>
> Often PRs fail when compiled with clang, get a head start on this by 
> making sure your PR compiles with clang:
> https://wiki.sphenix.bnl.gov/index.php?title=Tools#clang
>
> run cppcheck yourself (this is really dirt simple):
> https://wiki.sphenix.bnl.gov/index.php?title=Tools#Jenkins_Command_line
>
> There will always be oops'es and "I forgot this" but we all benefit if 
> we keep the load on jenkins as low as possible
>
> Thanks,
> Chris

--
*************************************************************

Christopher H. Pinkenburg       ;    pinkenburg AT bnl.gov
                                ;    http://www.phenix.bnl.gov/~pinkenbu

Brookhaven National Laboratory  ;    phone: (631) 344-5692
Physics Department Bldg 510 C   ;    fax:   (631) 344-3253
Upton, NY 11973-5000

*************************************************************