Skip to content

Add Windows Emscripten build of xeus-cpp to ci #647

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

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

mcbarton
Copy link
Collaborator

Description

Please include a summary of changes, motivation and context for this PR.

This PR adds a Windows Emscripten build of xeus-cpp to ci

Fixes # (issue)

Type of change

Please tick all options which are relevant.

  • Bug fix
  • New feature
  • Requires documentation updates

Testing

Please describe the test(s) that you added and ran to verify your changes.

Checklist

  • I have read the contribution guide recently

@mcbarton mcbarton marked this pull request as draft June 26, 2025 19:20
Copy link

codecov bot commented Jun 26, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 79.18%. Comparing base (2585d0d) to head (8e8716f).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #647   +/-   ##
=======================================
  Coverage   79.18%   79.18%           
=======================================
  Files           9        9           
  Lines        3853     3853           
=======================================
  Hits         3051     3051           
  Misses        802      802           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@mcbarton mcbarton force-pushed the Add-Windows-Emscripten-build-of-xeus-cpp-to-ci branch from bfac490 to f5be4c6 Compare June 30, 2025 16:54
@mcbarton
Copy link
Collaborator Author

For an Emscripten build of xeus-cpp to be possible on a Windows platform, the changes in this PR in xeus-cpp are required compiler-research/xeus-cpp#362

@mcbarton mcbarton force-pushed the Add-Windows-Emscripten-build-of-xeus-cpp-to-ci branch from f5be4c6 to b981887 Compare June 30, 2025 17:01
@mcbarton mcbarton marked this pull request as ready for review June 30, 2025 19:02
@mcbarton mcbarton marked this pull request as draft June 30, 2025 19:02
@mcbarton mcbarton force-pushed the Add-Windows-Emscripten-build-of-xeus-cpp-to-ci branch from b981887 to 8e8716f Compare July 1, 2025 22:17
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.

1 participant