Skip to content

Mahout-592: Update PULL_REQUEST_TEMPLATE to align with current workflow - #593

Merged
andrewmusselman merged 1 commit into
apache:mainfrom
guan404ming:pr-template
Oct 29, 2025
Merged

Mahout-592: Update PULL_REQUEST_TEMPLATE to align with current workflow#593
andrewmusselman merged 1 commit into
apache:mainfrom
guan404ming:pr-template

Conversation

@guan404ming

Copy link
Copy Markdown
Member

Purpose of PR

  • Clarified that PR title format is only needed "if related issue exists"
  • Consolidated testing/verification section
  • Made the template more concise while maintaining all essential checks

Linked Issues

Changes Made

  • Bug fix
  • New feature
  • Documentation update

Important ToDos

Please mark each with an "x"

A GitHub issue exists (if not, please create one) [https://github.com/apache/mahout/issues]

  • Title of PR is "Issue #XXXX: Brief Description of Changes" where XXXX is the GitHub issue number.
  • Created unit tests where appropriate
  • Added correct licenses on newly added files
  • Assigned GitHub issue to self
  • Added documentation in ScalaDocs/JavaDocs and to the website
  • Successfully built and ran all unit tests, verified that all tests pass locally

If all of these items are not yet complete, but you still feel it is appropriate to open a PR, please open it as a Draft PR instead.
Once all requirements are met, you can mark it as ready for review.

Breaking Changes

Does this PR introduce a breaking change?

  • Yes
  • No

Testing & Verification

Describe how you tested the changes.

  • Unit tests added
  • Manually tested

Checklist

  • The title follows the format "MAHOUT-XXXX Brief Description"
  • GitHub issue is created
  • Code follows ASF guidelines

@guan404ming
guan404ming force-pushed the pr-template branch 2 times, most recently from d258fce to 8119237 Compare October 29, 2025 07:38

@andrewmusselman andrewmusselman left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

@andrewmusselman
andrewmusselman merged commit 96de645 into apache:main Oct 29, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update PULL_REQUEST_TEMPLATE to align with current workflow

2 participants