Skip to content

Build target sphinx-cppinterop in ci docs job#371

Merged
mcbarton merged 1 commit intocompiler-research:mainfrom
mcbarton:Build-sphinx-documentation-ci
Dec 5, 2024
Merged

Build target sphinx-cppinterop in ci docs job#371
mcbarton merged 1 commit intocompiler-research:mainfrom
mcbarton:Build-sphinx-documentation-ci

Conversation

@mcbarton
Copy link
Copy Markdown
Collaborator

@mcbarton mcbarton commented Dec 4, 2024

When adding the docs jobs in the ci the other day it was forgotten that we need to build the target sphinx-cppinterop too. This PR fixes that.

@mcbarton mcbarton requested a review from vgvassilev December 4, 2024 21:42
@codecov
Copy link
Copy Markdown

codecov bot commented Dec 4, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 76.31%. Comparing base (111d0e5) to head (b229322).
Report is 4 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #371   +/-   ##
=======================================
  Coverage   76.31%   76.31%           
=======================================
  Files           8        8           
  Lines        3137     3137           
=======================================
  Hits         2394     2394           
  Misses        743      743           

Copy link
Copy Markdown
Contributor

@vgvassilev vgvassilev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm!

@mcbarton mcbarton merged commit 3ec1452 into compiler-research:main Dec 5, 2024
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.

2 participants