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 publish workflow #8564

Merged
merged 4 commits into from
Jan 13, 2025
Merged

Fix publish workflow #8564

merged 4 commits into from
Jan 13, 2025

Conversation

tautschnig
Copy link
Collaborator

@tautschnig tautschnig commented Jan 13, 2025

For an unknown reason mermaid-filter (or, really, one of its
dependencies) no longer work properly on Ubuntu 22.04. Using 24.04
addresses the problem. While at it, we also no longer need to overwrite
the puppeteer dependency as mermaid-filter has updated its mermaid-cli
dependency. No new release has been published just yet, so keep using
the git clone approach from 71d3b4e rather than installing a
release.

This entails updating the doxygen version we use by default, so please review commit-by-commit.

  • Each commit message has a non-empty body, explaining why the change was made.
  • n/a Methods or procedures I have added are documented, following the guidelines provided in CODING_STANDARD.md.
  • n/a The feature or user visible behaviour I have added or modified has been documented in the User Guide in doc/cprover-manual/
  • Regression or unit tests are included, or existing tests cover the modified code (in this case I have detailed which ones those are in the commit message).
  • n/a My commit message includes data points confirming performance improvements (if claimed).
  • My PR is restricted to a single feature or bugfix.
  • n/a White-space or formatting changes outside the feature-related changed lines are in commits of their own.

@tautschnig tautschnig self-assigned this Jan 13, 2025
@tautschnig tautschnig force-pushed the fix-doc-build branch 5 times, most recently from dc367b0 to 69f4f3b Compare January 13, 2025 14:42
@tautschnig tautschnig removed their assignment Jan 13, 2025
@tautschnig tautschnig marked this pull request as ready for review January 13, 2025 14:43
For an unknown reason mermaid-filter (or, really, one of its
dependencies) no longer work properly on Ubuntu 22.04. Using 24.04
addresses the problem. While at it, we also no longer need to overwrite
the puppeteer dependency as mermaid-filter has updated its mermaid-cli
dependency. No new release has been published just yet, so keep using
the `git clone` approach from 71d3b4e rather than installing a
release.

Fixes: diffblue#8555
No need to mix package managers.
This is the version shipped with Ubuntu 24.04.
Doxygen 1.9.8 detects more problems than earlier versions did.
Copy link

codecov bot commented Jan 13, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 78.95%. Comparing base (36b2335) to head (07cf0bb).
Report is 5 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #8564      +/-   ##
===========================================
+ Coverage    78.79%   78.95%   +0.15%     
===========================================
  Files         1730     1730              
  Lines       199160   198759     -401     
  Branches     18320    18314       -6     
===========================================
  Hits        156938   156938              
+ Misses       42222    41821     -401     

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

@tautschnig tautschnig merged commit 97c8624 into diffblue:develop Jan 13, 2025
39 of 40 checks passed
@tautschnig tautschnig deleted the fix-doc-build branch January 13, 2025 19:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants