Closed samjrholt closed 7 months ago
PR Description updated to latest commit (https://github.com/ubermag/discretisedfield/commit/680fd4145e3d7c8cc15e345180c9923301c25aa5)
⏱️ Estimated effort to review [1-5] | 1, because the PR involves a simple configuration change by removing a few lines in a YAML file. The change is straightforward and does not involve complex logic or multiple files. |
🧪 Relevant tests | No |
🔍 Possible issues | Possible Compatibility Issue: Removing the default Python version might cause issues if the system's default Python version is incompatible with some of the hooks or dependencies used in the pre-commit configuration. |
🔒 Security concerns | No |
Category | Suggestions |
Best practice |
Add a default Python version to ensure consistent pre-commit behavior across various environments.___ **Removing the default Python version from the pre-commit configuration might lead tounexpected behavior across different environments where the Python version is not explicitly managed. It's recommended to specify a default Python version that aligns with your project requirements or the minimum supported version.** [.pre-commit-config.yaml [1-3]](https://github.com/ubermag/discretisedfield/pull/522/files#diff-63a9c44a44acf85fea213a857769990937107cf072831e1a26808cfde9d096b9R1-R3) ```diff +default_language_version: + python: python3.x # Replace 'x' with the minimum supported version exclude: 'dev' repos: ``` |
Type
enhancement
Description
python3.8
) in the.pre-commit-config.yaml
, making the pre-commit hooks rely on the system's default Python version.Changes walkthrough
.pre-commit-config.yaml
Remove Default Python Version from Pre-commit Config
.pre-commit-config.yaml
configuration.