-
-
Notifications
You must be signed in to change notification settings - Fork 20
Description
CI Broken - Multiple Failures Detected
Summary
Automated CI health check detected 4 failing CI checks on the master branch. This PR health check was run on 2025-12-30 and revealed multiple pre-existing issues that need to be addressed.
Failed CI Jobs
1. Spell Check with Typos ❌
Error: Typo in documentation
Location: docs/src/index.md:79:11
Issue: paramters should be parameters
error: `paramters` should be `parameters`
--> ./docs/src/index.md:79:11
|
79 | spc_water_paramters = SPCFwParameters(rOH, ∠HOH, k_bond, k_angle)
| ^^^^^^^^^
Fix: Change spc_water_paramters to spc_water_parameters in line 79 of docs/src/index.md
2. Format Check ❌
Error: README.md fails JuliaFormatter v2 formatting check
Issue: The format check expects all files (including Markdown files) to pass JuliaFormatter validation
Details: The format check detected unformatted changes in README.md. While this was triggered by the health check PR's whitespace change, it indicates that the format check may be overly strict for Markdown files or that there's a configuration issue.
3. Dependency Resolution (alldeps, Julia 1.10) ❌
Error: Unsatisfiable requirements for OrdinaryDiffEq package
Critical Issue: Conflicting version constraints
ERROR: LoadError: Unsatisfiable requirements detected for package OrdinaryDiffEq [1dea7af3]:
OrdinaryDiffEq [1dea7af3] log:
├─possible versions are: 4.0.0-6.105.0 or uninstalled
├─restricted to versions 5-6 by project [21f58a0d], leaving only versions: 5.0.0-6.105.0
└─restricted to versions 6.45.0 by an explicit requirement — no versions left
Root Cause: The project has conflicting version constraints on OrdinaryDiffEq:
- Project.toml restricts to versions 5-6
- An explicit requirement somewhere restricts to exactly version 6.45.0
- These constraints cannot be simultaneously satisfied
Suggested Fix: Review and update the OrdinaryDiffEq version constraints in Project.toml and any test dependencies to ensure compatibility.
4. Documentation Build ❌
Error: Documentation build fails due to broken external links
Failed Links:
-
404 Error:
https://juliaarrays.github.io/StaticArrays.jl/stable/pages/quickstart/- This StaticArrays.jl documentation link returns 404
- Likely the documentation structure changed upstream
-
Connection Failed:
https://www2.mpip-mainz.mpg.de/~andrienk/journal_club/thermostats.pdf- curl failed with exit code 6 (couldn't resolve host)
- This external PDF reference may have been moved or removed
Additional Warning: Undocumented public symbols detected:
NBodySimulator.NBodySystem
Suggested Fixes:
- Update StaticArrays.jl documentation link to current URL
- Find an alternative source for the thermostats.pdf reference or remove it
- Add documentation for
NBodySystemor mark it as private if not intended to be public
CI Health Check Details
- PR: CI Health Check - Testing master branch #96
- Test Date: 2025-12-30
- CI Run Links:
Passing Tests ✅
The following tests passed successfully:
- Tests (Julia 1.x)
- Tests (lts)
- Tests (pre)
This indicates the core functionality is working, but CI infrastructure and documentation need attention.
Priority Assessment
- High Priority: OrdinaryDiffEq dependency conflict - blocks the alldeps test entirely
- Medium Priority: Documentation broken links - prevents docs deployment
- Low Priority: Typo in docs and format check - cosmetic but should be fixed
Next Steps
@ChrisRackauckas - The CI has multiple failures that need addressing. The most critical is the OrdinaryDiffEq dependency conflict in the alldeps test. Would you like me to investigate and propose fixes, or would you prefer to handle these issues?
This issue was automatically generated by an automated CI health check system.