-
Notifications
You must be signed in to change notification settings - Fork 49
small fixes in documentation #812
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
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #812 +/- ##
=======================================
Coverage 97.33% 97.33%
=======================================
Files 36 36
Lines 3495 3495
=======================================
Hits 3402 3402
Misses 93 93
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
@palday does the CI workflow for documentation deployment automatically install latest versions for the dependencies? |
The CI workflow will update the packages to the latest compatible versions as limited by the One thing I'm still looking at is how to integrate the deploy step from VitePress -- it's decidedly more complicated than the base Documenter.jl implementation, at least from my perspective. |
I agree. I can experiment with CI on my fork as a test. Can you help me with how to trigger a deployment ? |
Oh btw this PR has nothing to do with Vitepress |
@palday this PR is just small fixes to formatting. Do you think this is good to merge? |
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.
Looks good. Thanks for checking all these files.
Did behavior change? Did you add need features? No