RFR: 574: git-info should use info from commit notifications
Magnus Ihse Bursie
ihse at openjdk.java.net
Fri Feb 12 14:14:29 UTC 2021
git-info should use info from commit notifications to get the link to the review, not JBS (since not all PR have issues)
This required refactoring to move the hosted repo logic from cli/pr/Utils.java to where it can be accessed by all CLI tools. After some deliberating, I left a bunch of trivial wrappers in cli/pr/Utils.java -- otherwise this change would have touched *all* the files in cli/pr.
Testing: ad hoc testing of "git info" on some arbitrary Skara commits.
-------------
Commit messages:
- Use bot comment on forge for getReviewUrl instead of JBS
- Extract getReviewUrl
- Refactor so that `git info` can use hosted repository.
Changes: https://git.openjdk.java.net/skara/pull/1011/files
Webrev: https://webrevs.openjdk.java.net/?repo=skara&pr=1011&range=00
Issue: https://bugs.openjdk.java.net/browse/SKARA-574
Stats: 338 lines in 5 files changed: 199 ins; 124 del; 15 mod
Patch: https://git.openjdk.java.net/skara/pull/1011.diff
Fetch: git fetch https://git.openjdk.java.net/skara pull/1011/head:pull/1011
PR: https://git.openjdk.java.net/skara/pull/1011
More information about the skara-dev
mailing list