Skip to content
This repository has been archived by the owner on Nov 14, 2024. It is now read-only.

Fix pyodide runner #106

Merged
merged 16 commits into from
Sep 13, 2024
Merged

Fix pyodide runner #106

merged 16 commits into from
Sep 13, 2024

Conversation

vladsavelyev
Copy link
Member

@vladsavelyev vladsavelyev commented Sep 13, 2024

  • install pydantic==2.7.0 instead of 2.7.1 which is not available on Pyodide registry,
  • install pygments separately as for some reason it's not pulled as a dep
  • run multiqc.run with ClConfig
  • update pyodide itself *

Copy link

netlify bot commented Sep 13, 2024

Deploy Preview for multiqc ready!

Name Link
🔨 Latest commit 1e2f04d
🔍 Latest deploy log https://app.netlify.com/sites/multiqc/deploys/66e444b7f2b87a00080abbd5
😎 Deploy Preview https://deploy-preview-106--multiqc.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 92 (🟢 up 1 from production)
Accessibility: 97 (no change from production)
Best Practices: 100 (no change from production)
SEO: 92 (no change from production)
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.

@vladsavelyev vladsavelyev changed the title pyodite runner: install pydantic separately pyodide runner: install pydantic separately Sep 13, 2024
@vladsavelyev vladsavelyev marked this pull request as draft September 13, 2024 09:32
@vladsavelyev vladsavelyev marked this pull request as ready for review September 13, 2024 10:17
@vladsavelyev vladsavelyev changed the title pyodide runner: install pydantic separately Fix pyodide runner Sep 13, 2024
@vladsavelyev vladsavelyev merged commit 1b2deb6 into main Sep 13, 2024
3 of 5 checks passed
@vladsavelyev vladsavelyev deleted the runner branch September 13, 2024 13:57
@ewels
Copy link
Member

ewels commented Sep 15, 2024

Nice one!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants