RFR: 1547: Collect PR specific constants in single shared class
Zhao Song
zsong at openjdk.org
Fri Jan 27 00:06:58 UTC 2023
There are numerous string literals and constants duplicated and spread around different bots. This includes markers (used in PR body and comments) for various bot functionality as well as label names.
In this patch, I cleaned the constants in different bots according to this standard:
1. If a constant is used by multiple bots, it should be placed in bots.common.PullRequestConstants.
2. If a constant is used by different classes within a single bot, it should be placed in a single class within the bot, and other classes should import the constant.
3. If a constant is only used by one class within a single bot, it should be made private to that class.
Also, I have renamed the constants that were previously named in an inconsistent manner.
-------------
Commit messages:
- SKARA-1547
Changes: https://git.openjdk.org/skara/pull/1464/files
Webrev: https://webrevs.openjdk.org/?repo=skara&pr=1464&range=00
Issue: https://bugs.openjdk.org/browse/SKARA-1547
Stats: 344 lines in 40 files changed: 65 ins; 26 del; 253 mod
Patch: https://git.openjdk.org/skara/pull/1464.diff
Fetch: git fetch https://git.openjdk.org/skara pull/1464/head:pull/1464
PR: https://git.openjdk.org/skara/pull/1464
More information about the skara-dev
mailing list