search

cms-sw / cms-git-tools

CMS Git Helpers
34 stars 26 forks source link

Minor updates #123

Closed fwyzard closed 2 years ago

fwyzard commented 2 years ago

Update the usage and description of git cms-cherry-pick-pr. Do not show tags and branches in git cms-show-pr.

cmsbuild commented 2 years ago

A new Pull Request was created by @fwyzard (Andrea Bocci) for branch master.

@cmsbuild, @smuzaffar, @aandvalenzuela, @iarspider can you please review it and eventually sign? Thanks. @perrotta, @dpiparo, @rappoccio you are the release manager for this. cms-bot commands are listed here

cmsbuild commented 2 years ago

Pull request #123 was updated.

fwyzard commented 2 years ago

These are tools I wrote a couple of years ago, and then forgot about...

The idea is to simplify the process of looking at the commits that compose a PR (similar to what the GitHub page shows) and creating a new branch with those same commits, but targetting a different release cycle (or, patch release cycle).

fwyzard commented 2 years ago

please test

smuzaffar commented 2 years ago

+externals looks good to me. @fwyzard if you are happy with changes then I can merge it and open a cmsdist PR to include it.

cmsbuild commented 2 years ago

This pull request is fully signed and it will be integrated in one of the next master IBs after it passes the integration tests. This pull request will now be reviewed by the release team before it's merged. @perrotta, @dpiparo, @rappoccio (and backports should be raised in the release meeting by the corresponding L2)

cmsbuild commented 2 years ago

Pull request #123 was updated.

fwyzard commented 2 years ago

sorry @smuzaffar, I just found another bug... the changes should complete now.

fwyzard commented 2 years ago

please test

smuzaffar commented 2 years ago

+externals

cmsbuild commented 2 years ago

This pull request is fully signed and it will be integrated in one of the next master IBs after it passes the integration tests. This pull request will now be reviewed by the release team before it's merged. @perrotta, @dpiparo, @rappoccio (and backports should be raised in the release meeting by the corresponding L2)

cmsbuild commented 2 years ago

+1

Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-816d23/28175/summary.html COMMIT: 3e51ecc7855fbc66dfd3457cc56069d8bbb3ff27 CMSSW: CMSSW_12_6_X_2022-10-11-1100/el8_amd64_gcc10 User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week0/cms-sw/cms-git-tools/123/28175/install.sh to create a dev area with all the needed externals and cmssw changes.

Comparison Summary

@slava77 comparisons for the following workflows were not done due to missing matrix map:

Summary: