Skip to content

read the docs deployment preview #361

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

Open
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.

Fixes # (issue)

Type of change

Please tick all options which are relevant.

  • Bug fix
  • New feature
  • Added/removed dependencies
  • Required documentation updates

@codecov-commenter
Copy link

codecov-commenter commented Jun 27, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.78%. Comparing base (7a3a24b) to head (2b7414a).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #361   +/-   ##
=======================================
  Coverage   81.78%   81.78%           
=======================================
  Files          20       20           
  Lines         950      950           
  Branches       87       87           
=======================================
  Hits          777      777           
  Misses        173      173           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@mcbarton mcbarton force-pushed the Read-the-docs-xeus-cpp-preview branch 4 times, most recently from 7d1744e to 58148cd Compare June 29, 2025 10:52
@mcbarton mcbarton closed this Jun 30, 2025
@mcbarton mcbarton reopened this Jun 30, 2025
@mcbarton mcbarton closed this Jun 30, 2025
@mcbarton mcbarton reopened this Jun 30, 2025
@mcbarton mcbarton closed this Jun 30, 2025
@mcbarton mcbarton reopened this Jun 30, 2025
@mcbarton mcbarton force-pushed the Read-the-docs-xeus-cpp-preview branch from 3097459 to 2b7414a Compare June 30, 2025 17:35
@mcbarton mcbarton changed the title [wip] read the docs deployment preview read the docs deployment preview Jun 30, 2025
@mcbarton
Copy link
Collaborator Author

@vgvassilev @SylvainCorlay @anutosh491 I couldn't get the auto commenter to work as I wanted from a fork. Apparently its the way Github is designed for security concerns. But once this PR is in, and the readthedocs workflow has finished for a particular PR, you can preview the xeus-cpp deployment for that PR by going to https://xeus-cpp--${PR-number}.org.readthedocs.build/en/${PR-number}/xeus-cpp/lab/index.html (obviously replacing ${PR-number} with the PR number of the PR).

In this case it would be https://xeus-cpp--361.org.readthedocs.build/en/361/xeus-cpp/lab/index.html

This PR is ready for review.

@mcbarton mcbarton marked this pull request as ready for review June 30, 2025 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.

2 participants