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

Add MyST based documentation testing #7186

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Add MyST based documentation testing #7186

wants to merge 5 commits into from

Conversation

philippjfr
Copy link
Member

An attempt to add a MyST based documentation build to ensure that Markdown output can be parsed.

.github/workflows/test.yaml Outdated Show resolved Hide resolved
doc/_plugins/pyodide.mjs Outdated Show resolved Hide resolved
pixi.toml Outdated Show resolved Hide resolved
pixi.toml Outdated Show resolved Hide resolved
Co-authored-by: Simon Høxbro Hansen <simon.hansen@me.com>
- uses: holoviz-dev/holoviz_tasks/pixi_install@pixi
with:
environments: ${{ matrix.environment }}
- name: Test Type
Copy link
Member

@hoxbro hoxbro Aug 26, 2024

Choose a reason for hiding this comment

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

Suggested change
- name: Test Type
- name: Test MyST

Copy link

codecov bot commented Aug 26, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 82.20%. Comparing base (3697335) to head (9fe061e).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #7186      +/-   ##
==========================================
+ Coverage   72.23%   82.20%   +9.97%     
==========================================
  Files         329      330       +1     
  Lines       49342    49343       +1     
==========================================
+ Hits        35640    40564    +4924     
+ Misses      13702     8779    -4923     

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

@philippjfr
Copy link
Member Author

Output is somewhat helpful so far but it doesn't seem to have caught the issue that's currently breaking the doc build

@philippjfr philippjfr added this to the v1.6.0 milestone Sep 9, 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