A PR may be associated with multiple issues, so writing Issue No. in PR titile 
may 
be miscellaneous.



------------------ ???????? ------------------
??????:&nbsp;"Yichao Yang"<[email protected]&gt;;
????????:&nbsp;2020??7??8??(??????) ????9:29
??????:&nbsp;"dev"<[email protected]&gt;;

????:&nbsp;[Vote] Pull Request Title Specification



Hi all,


After referring to the practices of other open source communities on the title 
of pull request, we propose two pre selection schemes for dolphin scheduler.


First??

Specification:

[`Open Source Project Name`-`Issue No`][`Module Name`] `Pull Request 
Description`

example??

[DS-3333][server] Fix the xxx exception

[DS-3333][server] Implement xxx

...




Second??

Specification:

[`Pull Request Type`-`Issue No`][`Module Name`] `Pull Request Description`

example??

[Fix-3333][server] Fix the xxx exception

[Feature-3333][server] Implement xxx

...

The Pull Request type is shown in the screenshot in [1].




If you agree with the first one as the title of the pull request, reply 1, and 
the second one reply 2. If there is no other suggestions after 7 days, we will 
choose the first one.




--------------------------------------------------------




???????????????????????????? Pull Request ???????????????????? DolphinScheduler 
??????????????????????????


????????

Specification:

[`Open Source Project Name`-`Issue No`][`Module Name`] `Pull Request 
Description`

example??

[DS-3333][server] Fix the xxx exception

[DS-3333][server] Implement xxx

...




????????

Specification:

[`Pull Request Type`-`Issue No`][`Module Name`] `Pull Request Description`

example??

[Fix-3333][server] Fix the xxx exception

[Feature-3333][server] Implement xxx

...

Pull Request Type ?????? [1] ????????????




??????????????????Pull request 
????????????1??????????????2????????????????????????????7?????????????????????? 
Pull Request ??????




[1] Pull Request Type: 
https://github.com/apache/incubator-dolphinscheduler-website/pull/147




Best,

Yichao Yang

Reply via email to