-
Notifications
You must be signed in to change notification settings - Fork 72
fixes for uml github action #446
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR updates the project configuration to support UML workflows by refining optional dependencies used for documentation generation.
- Adds pylint (providing pyreverse) to the docs dependencies in pyproject.toml.
- Updates the makefile to load the docs optional dependencies.
Files not reviewed (1)
- Makefile: Language not supported
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #446 +/- ##
=======================================
Coverage 94.40% 94.40%
=======================================
Files 31 31
Lines 1985 1985
=======================================
Hits 1874 1874
Misses 111 111 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems good. Did the manual trigger work?
Just tried that and no luck https://github.com/pymc-labs/CausalPy/actions/runs/14260986935/job/39972351813 But trying a couple of things... |
No luck on a manual run https://github.com/pymc-labs/CausalPy/actions/runs/14261271099/job/39973210151 Seems like |
Maybe try python -m pyreverse |
So that doesn't work locally, so not sure it would help. But I've added a debug step
which results in |
That last commit worked. it seems to require explicit installation of |
pylint
(which containspyreverse
) in the optional docs dependencies📚 Documentation preview 📚: https://causalpy--446.org.readthedocs.build/en/446/