guan404ming commented on code in PR #1038: URL: https://github.com/apache/mahout/pull/1038#discussion_r2785391154
########## .github/PULL_REQUEST_TEMPLATE: ########## @@ -16,18 +12,10 @@ - [ ] CI/CD pipeline - [ ] Other -### Breaking Changes -<!-- Does this PR introduce a breaking change? --> -- [ ] Yes -- [ ] No +### Why + +<!-- Why is this change needed? --> -### Checklist -<!-- Please mark each item with an "x" when complete --> -<!-- If not all items are complete, please open this as a **Draft PR**. -Once all requirements are met, mark as ready for review. --> +### How -- [ ] Added or updated unit tests for all changes -- [ ] Added or updated documentation for all changes -- [ ] Successfully built and ran all unit tests or manual tests locally -- [ ] PR title follows "MAHOUT-XXX: Brief Description" format (if related to an issue) -- [ ] Code follows ASF guidelines Review Comment: Thanks for the notification There is no those checklist in Airflow and we got some ci to check ASF header in each files so I think we could safely remove them. -- 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]
