roedoejet / g2p

Grapheme-to-Phoneme transductions that preserve input and output indices, and support cross-lingual g2p!
https://g2p-studio.herokuapp.com
Other
119 stars 26 forks source link

ci: revamp how we publish documentation #342

Closed joanise closed 3 months ago

joanise commented 3 months ago

There were several bugs that needed fixing in the docs publication workflow.

New model:

This PR will require a manual push to gh-pages before it's safe to merge, but afterwards docs updates will be automatic.

github-actions[bot] commented 3 months ago
CLI load time: 0:00.05
PR head 194b8e839ef7adba1a8715133351190a3c1d6d22
Imports that take more than 0.1 s:
import time: self [us] | cumulative | imported package
joanise commented 3 months ago

BTW, the two docs build are not as DRY as ideal, but as I was writing the code to pass the tag and aliases from publish to docs, it turned out that was more complicated and more lines of code than just copy-pasting the checkout/python/dependencies steps from docs.yml to the deploy-docs job in pythonpublish.yml, so I decided that was not worth it.

codecov[bot] commented 3 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 93.23%. Comparing base (cbfd608) to head (194b8e8). Report is 4 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #342 +/- ## ========================================== - Coverage 93.44% 93.23% -0.22% ========================================== Files 18 18 Lines 2334 2334 Branches 516 516 ========================================== - Hits 2181 2176 -5 - Misses 86 90 +4 - Partials 67 68 +1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.