kaijchen commented on issue #551: URL: https://github.com/apache/incubator-uniffle/issues/551#issuecomment-1418439241
I would suggest omit issue id in the title, and just add it in the PR description. ``` ### Why this change needed? Fix #123 ``` The PR title could be `[module][type] title`. Modules: * `coordinator` * `server` * `client` * `operator` * `*`: all * etc Types: * `feature`: (new feature for the user, not a new feature for build script) * `fix`: (bug fix for the user, not a fix to a build script) * `docs`: (changes to the documentation) * `style`: (formatting, missing semi colons, etc; no production code change) * `refactor`: (refactoring production code, eg. renaming a variable) * `test`: (adding missing tests, refactoring tests; no production code change) * `chore`: (updating grunt tasks etc; no production code change) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
