apache / parquet-java

Apache Parquet Java
https://parquet.apache.org/
Apache License 2.0
2.65k stars 1.41k forks source link

GH-2939: Update PULL_REQUEST_TEMPLATE #2940

Closed wgtmac closed 4 months ago

wgtmac commented 4 months ago

Rationale for this change

The original PR template is a little bit out of date, especially after moving from JIRA to Github issue.

What changes are included in this PR?

Update PULL_REQUEST_TEMPLATE to remove unnecessary check boxes and follow what Apache Arrow does.

Are these changes tested?

This is not a code change.

Are there any user-facing changes?

No.

Closes #2939

wgtmac commented 4 months ago

@rok @gszadovszky @Fokko @julienledem Would you mind taking a look?

wgtmac commented 4 months ago

Thanks for the suggestion! I've simplified the wording and removed unimportant comments. PTAL. @Fokko @gszadovszky

wgtmac commented 4 months ago

@Fokko Gentle ping :)

wgtmac commented 4 months ago

Thanks @Fokko for merging!