[CI] Add `pre-commit` hook `name-tests-test` for Python test files - Githubissues

apache / sedona

A cluster computing framework for processing large-scale geospatial data

https://sedona.apache.org/

Apache License 2.0

1.96k stars 692 forks source link

[CI] Add `pre-commit` hook `name-tests-test` for Python test files #1646

Closed jbampton closed 3 weeks ago

jbampton commented 1 month ago

https://github.com/pre-commit/pre-commit-hooks?tab=readme-ov-file#name-tests-test

--pytest-test-first: ensure tests match test_.*.py

https://pre-commit.com/#regular-expressions

Did you read the Contributor Guide?

Yes, I have read the Contributor Rules and Contributor Development Guide

Is this PR related to a JIRA ticket?

No this is a CI update. The PR name follows the format [CI] my subject

What changes were proposed in this PR?

Added another pre-commit hook from the official pre-commit team.

Good to have file naming standards.

How was this patch tested?

Ran pre-commit run --all-files

Did this PR include necessary documentation updates?

No, this PR does not affect any public API so no need to change the documentation.

jiayuasu commented 4 weeks ago

@jbampton Can you fix this PR?

jbampton commented 4 weeks ago

Fixed now @jiayuasu