This PR addresses the issue with the pre-commit hook and the poetry GitHub action build check in the Deep-Learning-Playground repository. The issue was causing a failed build check even when no poetry dependencies were touched. The root cause of the issue was identified and resolved by making the following changes:
Updated the configuration of the pre-commit hook for poetry.lock in the .pre-commit-config.yaml file to ensure it correctly checks if the poetry.lock file is up-to-date with the pyproject.toml file.
Checked and updated the configuration of the poetry GitHub action build check in the .github/workflows/build.yml file to ensure it correctly checks if the poetry.lock file is up-to-date with the pyproject.toml file.
Summary of Changes
Modified the .pre-commit-config.yaml file to update the configuration of the pre-commit hook for poetry.lock.
Modified the .github/workflows/build.yml file to update the configuration of the poetry GitHub action build check.
These changes ensure that the pre-commit hook and the poetry GitHub action build check function as intended, preventing a failed build check when no poetry dependencies are touched.
Please review and merge this PR to resolve the issue.
Fixes #894.
To checkout this PR branch, run the following command in your terminal:
git checkout sweep/fix-pre-commit-hook
To get Sweep to edit this pull request, leave a comment below or in the code. Leaving a comment in the code will only modify the file but commenting below can change the entire PR.
Description
This PR addresses the issue with the pre-commit hook and the poetry GitHub action build check in the Deep-Learning-Playground repository. The issue was causing a failed build check even when no poetry dependencies were touched. The root cause of the issue was identified and resolved by making the following changes:
Summary of Changes
These changes ensure that the pre-commit hook and the poetry GitHub action build check function as intended, preventing a failed build check when no poetry dependencies are touched.
Please review and merge this PR to resolve the issue.
Fixes #894.
To checkout this PR branch, run the following command in your terminal:
To get Sweep to edit this pull request, leave a comment below or in the code. Leaving a comment in the code will only modify the file but commenting below can change the entire PR.