Skip to content
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

[fix] Moving from Jekyll to Sphinx. RTD theme is safe! #515

Merged
merged 1 commit into from
Sep 9, 2024

Conversation

lpascal-ledger
Copy link
Contributor

New theme pushed in #514 was utterly broken.
And I could not easily figure out how the Ruby / Jekyll environment was working (I could not make the theme layouts to be used while serving the site locally).
As pretty much nobody in the team masters Ruby, this PR moves the site generating from Jekyll to Sphinx (yay Python).

I used the markdown plugin to minimize the doc .md file modification, and the RTD theme is back on line!

@codecov-commenter
Copy link

codecov-commenter commented Sep 6, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 44.91%. Comparing base (8c11b57) to head (4e44938).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #515   +/-   ##
=======================================
  Coverage   44.91%   44.91%           
=======================================
  Files         120      120           
  Lines       12067    12067           
  Branches      966      966           
=======================================
  Hits         5420     5420           
  Misses       6312     6312           
  Partials      335      335           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@lpascal-ledger lpascal-ledger merged commit 2fef8ab into master Sep 9, 2024
38 checks passed
@lpascal-ledger lpascal-ledger deleted the fix/doc branch September 9, 2024 07:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants