• Closes: [Documentation] Update PR template #33619

⚠️ GitHub issue #33619 has been automatically assigned in GitHub to PR creator.

⚠️ GitHub issue #33619 has no components, please add labels for components.

Looks good to me! Thanks for making this change, @assignUser!

Following up on the discussion about stripping comments from the pull request description when merging, I believe adding the following around line 583 of merge_arrow_pr.py should work:

if self.body is not None:
    # Remove comments (i.e. <-- comment -->) from the pull request description.
    body = re.sub(r"<--.*-->", "", self.body)
    # avoid github user name references by inserting a space after @ 
    body = re.sub(r"@(\w+)", "@ \\1", body)
    commit_message_chunks.append(body)

Of course, there is a chance that a contributor might intend to include a literal use of "<-- comment -->" somewhere in their pull request description, but that's seems like a fairly uncommon scenario.

Following up on the discussion about stripping comments from the pull request description when merging, I believe adding the following around line 583 of merge_arrow_pr.py should work:

Short term that would be good to do to avoid those long noisy commit messages. Longer term, IMO we should discuss using only the title of the PR.

Whoops, sorry, the linter was failing on the code line in archery .. This will be fixed by #15067

Benchmark runs are scheduled for baseline = 7d3bca3 and contender = 7e6bcd1. 7e6bcd1 is a master commit associated with this PR. Results will be available as each benchmark for each run completes.
Conbench compare runs links:
[Finished ⬇️0.0% ⬆️0.0%] ec2-t3-xlarge-us-east-2
[Finished ⬇️0.39% ⬆️0.0%] test-mac-arm
[Finished ⬇️0.0% ⬆️0.0%] ursa-i9-9960x
[Finished ⬇️0.22% ⬆️0.03%] ursa-thinkcentre-m75q
Buildkite builds:
[Finished] 7e6bcd1b ec2-t3-xlarge-us-east-2
[Finished] 7e6bcd1b test-mac-arm
[Finished] 7e6bcd1b ursa-i9-9960x
[Finished] 7e6bcd1b ursa-thinkcentre-m75q
[Finished] 7d3bca39 ec2-t3-xlarge-us-east-2
[Finished] 7d3bca39 test-mac-arm
[Finished] 7d3bca39 ursa-i9-9960x
[Finished] 7d3bca39 ursa-thinkcentre-m75q
Supported benchmarks:
ec2-t3-xlarge-us-east-2: Supported benchmark langs: Python, R. Runs only benchmarks with cloud = True
test-mac-arm: Supported benchmark langs: C++, Python, R
ursa-i9-9960x: Supported benchmark langs: Python, R, JavaScript
ursa-thinkcentre-m75q: Supported benchmark langs: C++, Java