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

Try reverting the only changes to build_doc since the break #429

Closed
wants to merge 15 commits into from

Conversation

PProfizi
Copy link
Contributor

@PProfizi PProfizi commented Aug 25, 2022

The Documentation check passes but the documentation artifact is actually empty (easily verified with its small size of 48 kB, should be more than 100MB).
The last known Documentation generation job to work was here on August 22.
Was rerun to check for unseen change in a dependency, it still passed.

Seems to have started with bump of Graphviz to 0.20.1 here. Cannot find an occurrence before that.

Two problems here:

the documentation check did not see something was wrong with the doc (artifact too small / step too short)
no error was raised although something went obviously wrong

@PProfizi PProfizi added bug Something isn't working CI/CD Related to CI/CD labels Aug 25, 2022
@PProfizi PProfizi self-assigned this Aug 25, 2022
@codecov
Copy link

codecov bot commented Aug 25, 2022

Codecov Report

Merging #429 (b767277) into master (3ca6c70) will decrease coverage by 0.07%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #429      +/-   ##
==========================================
- Coverage   86.39%   86.31%   -0.08%     
==========================================
  Files          61       61              
  Lines        6717     6717              
==========================================
- Hits         5803     5798       -5     
- Misses        914      919       +5     

@PProfizi PProfizi closed this Sep 12, 2022
@PProfizi PProfizi deleted the fix/doc_generation branch November 28, 2024 14:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working CI/CD Related to CI/CD
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant