Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #1484 +/- ##
==========================================
- Coverage 91.99% 91.97% -0.02%
==========================================
Files 87 88 +1
Lines 18268 18376 +108
==========================================
+ Hits 16805 16902 +97
- Misses 1463 1474 +11 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
j178
left a comment
There was a problem hiding this comment.
Thanks! Overall looks good. I added a few comments, and we’ll need to update the docs/languages.md file.
📦 Cargo Bloat ComparisonBinary size change: +0.44% (22.5 MiB → 22.6 MiB) Expand for cargo-bloat outputHead Branch ResultsBase Branch Results |
|
To mitigate the race condition issue with |
|
Can we skip |
|
Ok |
Add remote hook test Add `haskell-setup` ci tweak tweak tweak clippy fix tweak tweak fix unimplemented test tweak
|
The haskell-actions/setup is excessively slow,especially macOS |
# Conflicts: # .github/workflows/ci.yml
There was a problem hiding this comment.
Pull request overview
This pull request adds support for the Haskell language to prek, resolving issue #1445. The implementation installs Haskell hooks via Cabal and runs configured entries using the system's cabal and ghc installations.
Changes:
- Implemented Haskell language support with
cabal v2-installfor dependency management - Added comprehensive test coverage including local hooks, additional dependencies, and remote hooks
- Updated CI/CD pipeline to run Haskell tests with GHC 9.8.2 and Cabal 3.16.1.0
Reviewed changes
Copilot reviewed 9 out of 9 changed files in this pull request and generated 9 comments.
Show a summary per file
| File | Description |
|---|---|
docs/languages.md |
Updated documentation to mark Haskell as supported with usage instructions |
crates/prek/src/languages/haskell.rs |
New implementation of Haskell language support using Cabal for installation and execution |
crates/prek/src/languages/mod.rs |
Integrated Haskell language into the language registry |
crates/prek/tests/languages/haskell.rs |
Added comprehensive tests for local hooks, additional dependencies, and remote hooks |
crates/prek/tests/languages/main.rs |
Added haskell module to test suite |
crates/prek/tests/languages/unimplemented.rs |
Updated to use R language as example of unimplemented language |
crates/prek-consts/src/env_vars.rs |
Added PREK_INTERNAL__SKIP_CABAL_UPDATE environment variable for testing |
.github/workflows/ci.yml |
Added Haskell to CI matrix with GHC/Cabal installation and caching on macOS |
.config/nextest.toml |
Added lang-haskell test profile configuration |
|
Thank you! |
resolve #1445