Skip to content

Update project pull request template#7934

Merged
raboof merged 1 commit intoapache:developfrom
semioticrobotic:patch-1
Sep 27, 2025
Merged

Update project pull request template#7934
raboof merged 1 commit intoapache:developfrom
semioticrobotic:patch-1

Conversation

@semioticrobotic
Copy link
Member

This commit streamlines the project's GitHub pull request template. Primarily, it removes white space between bullet items, which add unnecessary visual bulk to new pull requests. It also rewords a code comment and removes one that seems to reference deprecated systems.

For all changes:

  • Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?
  • Has your PR been rebased against the latest commit within the target branch (typically develop)?
  • Is your initial contribution a single, squashed commit?
  • Does gradlew build run cleanly?
  • Have you written or updated unit tests to verify your changes?
  • If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

This commit streamlines the project's GitHub pull request template. Primarily, it removes white space between bullet items, which add unnecessary visual bulk to new pull requests. It also rewords a code comment and removes one that seems to reference deprecated systems.
@semioticrobotic
Copy link
Member Author

A small change I hope saves a bit of time, screen real estate, and eye strain. Thanks for your consideration @JinwooHwang, @raboof, @nitin-amp, @sboorlagadda, and anyone else who may be able to take a look.

@sboorlagadda
Copy link
Member

I am good with this proposal. @semioticrobotic. We generally use a JIRA ticket for every commit. But I am good with out also. Let others chime in.

@semioticrobotic
Copy link
Member Author

I am good with this proposal. @semioticrobotic. We generally use a JIRA ticket for every commit. But I am good with out also. Let others chime in.

Noted! Thanks, @sboorlagadda!

@raboof raboof merged commit 77014c9 into apache:develop Sep 27, 2025
17 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.

3 participants