LGTM. Thanks, Yunze
> 2022年8月17日 11:15,Yu <li...@apache.org> 写道: > > Hi team, > > For PIP 198: Standardize PR Naming Convention using GitHub Actions [1] > > How to define [type] and [scope]? Do these abbreviations LGTY? > > *[Guide] Pulsar Pull Request Naming Convention* [2] contains everything > about the definition. Feel free to check and comment! > > ~~~~~~~~~~ > > TL;DR > > PR title format: [type][scope] Summary [3] > > ~~~~~~~~~~ > > [type] > > 1. Definition: what actions do you take? > > 2. It must be one of the following: > - feat (abbr for "feature") > - improve > - fix > - cleanup > - refactor > - revert > > ~~~~~~~~~~ > > [scope] > > 1. Definition: where do you make changes? > > 2. It must be one of the following: > - admin (changes to pulsar-admin, REST API, Java admin API) > - broker > - io > - deploy > - dep (abbr for dependency) > - fcn (abbr for function) > - monitor > - pkg (abbr for package) > - proxy > - schema > - sec (abbr for security) > - sql > - ts (abbr for tiered storage) > - tool > - txn (abbr for transaction) > > - java (changes to Java client) > - cpp (changes to C++ client) > - py (changes to Python client) > - ws (changes to WebSocket) > - rest (changes to REST) > > - test > - ci > - workflow > - build > - misc (abbr for miscellaneous) > > - doc > - blog > - site (abbr for website) > > ~~~~~~~~~~ > > Besides, many developers have different opinions on the following aspects. > What's your writing preference? > > - Submit breaking changes > [feat][broker]! Support xx > > - Submit PIP changes > [feat][broker] PIP-198: Support xx > > - Cherry pick changes [4] > Choice A: [fix][broker][branch-2.9] xxx > Choice B: [fix][broker] xxx. And add "cherry pick xxx to branch-2.9" in the > PR description. > > ~~~~~~~~~~ > > Feel free to comment and make your voice heard. Go vote! Thank you! > > [1] > https://docs.google.com/document/d/1sJlUNAHnYAbvu9UtEgCrn_oVTnVc1M5nHC19x1bFab4/edit > [2] https://lists.apache.org/thread/90rcjf1dv0fbkb5hm31kmgr65fj0nfnn > [3] > https://docs.google.com/document/d/1d8Pw6ZbWk-_pCKdOmdvx9rnhPiyuxwq60_TrD68d7BA/edit?pli=1#bookmark=id.y8943h392zno > [4] > https://docs.google.com/document/d/1d8Pw6ZbWk-_pCKdOmdvx9rnhPiyuxwq60_TrD68d7BA/edit?pli=1#bookmark=kix.849jztd92uk7 > > Yu