Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #1519 +/- ##
==========================================
+ Coverage 91.98% 92.02% +0.03%
==========================================
Files 88 89 +1
Lines 18374 18500 +126
==========================================
+ Hits 16901 17024 +123
- Misses 1473 1476 +3 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
update docs and ci tweak tweak Addd lang-julia profile order
📦 Cargo Bloat ComparisonBinary size change: +0.00% (22.6 MiB → 22.6 MiB) Expand for cargo-bloat outputHead Branch ResultsBase Branch Results |
|
I haven't added the remote hook test yet. Once you've forked the repository (https://github.com/fllesser/julia-hooks), I will push the test cases @j178 |
|
Forked https://github.com/prek-test-repos/julia-hooks! Also invited you as a member to prek-test-repos. |
Co-authored-by: Jo <10510431+j178@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
This PR adds support for the Julia language to prek, implementing a new language handler that allows hooks written in Julia to be installed and executed. The implementation follows established patterns from other language implementations like Haskell and Lua.
Changes:
- Added Julia language support with isolated environment management using Julia's
Pkgpackage manager - Integrated Julia throughout the codebase including schema, configuration, documentation, tests, and CI/CD
- Fixed alphabetical ordering of language enums (moved Pygrep before Swift)
Reviewed changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
prek.schema.json |
Added Julia to language schema and default_language_version in alphabetical order |
docs/languages.md |
Added Julia documentation describing installation, features, and limitations |
docs/cli.md |
Updated CLI documentation to include Julia in language list |
crates/prek/src/languages/julia.rs |
New Julia language implementation with install, health check, and run methods |
crates/prek/src/languages/mod.rs |
Integrated Julia language handler into the languages module |
crates/prek/src/config.rs |
Added Julia to Language enum with proper alphabetical ordering |
crates/prek/tests/languages/julia.rs |
Comprehensive test suite for Julia hooks covering local, remote, script file, and dependency scenarios |
crates/prek/tests/languages/main.rs |
Added Julia test module |
.github/workflows/ci.yml |
Added Julia installation step and version configuration for CI |
.config/nextest.toml |
Added Julia test profile configuration |
|
Thanks! |
resolve #1446