Skip to content

Commit a1acfb5

Browse files
committed
feat: re-enable typos hook in CI with fresh cache invalidation
Phase 4: Typos Hook Re-activation - Re-enabled typos hook in CI workflow after cache invalidation - Removed SKIP=typos environment variable - Enhanced typos.toml configuration should handle all spelling issues - Fresh cache keys ensure no legacy tool conflicts Cache Invalidation Strategy Applied: ✅ Updated terraform-tools cache key to v2-cache-invalidation ✅ Updated pre-commit hooks cache key to v2-cache-invalidation ✅ Added explicit pre-commit clean step ✅ Enhanced logging for debugging Local Development: ✅ Comprehensive typos.toml with 15+ misspelling patterns ✅ Enhanced pre-commit hook configuration ✅ Complete spell-check documentation in .github/SPELL_CHECK.md This should resolve the persistent typo errors in CI while maintaining robust spell-checking for both local development and CI environments.
1 parent 63e0360 commit a1acfb5

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

.github/workflows/pre-commit.yml

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -95,9 +95,9 @@ jobs:
9595
- name: Run pre-commit on all files (push to master)
9696
if: github.event_name == 'push' && github.ref == 'refs/heads/master'
9797
run: |
98-
# Temporarily skip typos hook in CI until cache issues are resolved
99-
# Typos hook still works locally via enhanced typos.toml configuration
100-
SKIP=typos pre-commit run --all-files
98+
# Re-enabling typos hook with fresh cache invalidation
99+
# Enhanced typos.toml configuration should handle all spelling issues
100+
pre-commit run --all-files
101101
102102
- name: Run pre-commit on changed files (pull request)
103103
if: github.event_name == 'pull_request'
@@ -109,9 +109,9 @@ jobs:
109109
if [ -n "$CHANGED_FILES" ]; then
110110
echo "Running pre-commit on changed files:"
111111
echo "$CHANGED_FILES"
112-
# Temporarily skip typos hook in CI until cache issues are resolved
113-
# Typos hook still works locally via enhanced typos.toml configuration
114-
SKIP=typos pre-commit run --files $CHANGED_FILES
112+
# Re-enabling typos hook with fresh cache invalidation
113+
# Enhanced typos.toml configuration should handle all spelling issues
114+
pre-commit run --files $CHANGED_FILES
115115
else
116116
echo "No relevant files changed, skipping pre-commit checks"
117117
fi

0 commit comments

Comments
 (0)