Skip to content

[FIX] Docs build failures#326

Merged
DESm1th merged 1 commit intomasterfrom
DESm1th-patch-2
Mar 10, 2022
Merged

[FIX] Docs build failures#326
DESm1th merged 1 commit intomasterfrom
DESm1th-patch-2

Conversation

@DESm1th
Copy link
Copy Markdown
Contributor

@DESm1th DESm1th commented Mar 3, 2022

Have to explicitly pin docutils below version 0.18.0 or the circleci build_docs step fails

Have to explicitly pin docutils below version 0.18.0 or the circleci build_docs step fails
@auto-assign auto-assign bot requested review from benselby, josephmje and jskocic March 3, 2022 21:47
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 3, 2022

Codecov Report

Merging #326 (860dd58) into master (6a0d16e) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #326   +/-   ##
=======================================
  Coverage   30.83%   30.83%           
=======================================
  Files          63       63           
  Lines        9501     9501           
=======================================
  Hits         2930     2930           
  Misses       6571     6571           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6a0d16e...860dd58. Read the comment docs.

@josephmje
Copy link
Copy Markdown
Contributor

Thanks for this fix @DESm1th! I was wondering what do you think about migrating our CI to GitHub Actions?

@DESm1th
Copy link
Copy Markdown
Contributor Author

DESm1th commented Mar 4, 2022

I'm not sure if it would be worth it yet. It's not as mature or featureful as circleci yet, and does some weird things with caching that would probably make our jobs slower atm.

@DESm1th DESm1th merged commit 4d3096d into master Mar 10, 2022
@DESm1th DESm1th deleted the DESm1th-patch-2 branch March 10, 2022 17:42
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