Skip to content

Conversation

pinin4fjords
Copy link
Collaborator

@pinin4fjords pinin4fjords commented Jul 7, 2025

Just mocking up how we might configure this to build for Studios. Works with DinD, test with cr.seqera.io/scidev/nextflow-training-studios

Copy link

netlify bot commented Jul 7, 2025

Deploy Preview for nextflow-training ready!

Name Link
🔨 Latest commit 239f40f
🔍 Latest deploy log https://app.netlify.com/projects/nextflow-training/deploys/6896290b19b2ee0008d27442
😎 Deploy Preview https://deploy-preview-629--nextflow-training.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

@pinin4fjords pinin4fjords marked this pull request as draft July 7, 2025 17:42
- Switch to cr.seqera.io/public/data-studio-vscode base image
- Change workspace path from /workspace to /workspaces (Codespaces convention)
- Add comprehensive VS Code settings for dark mode, auto-open terminal, and training directory
- Pre-install nf-core extension pack for immediate availability
- Patch init script to open VS Code in training directory by default
- Add conservative JVM tuning to prevent Nextflow core dumps
- Improve setup script with better Nextflow initialization and timing
- Use heredoc syntax for cleaner settings.json formatting
- Add proper variable usage with ${workspaceFolder} for terminal directory
@@ -0,0 +1,58 @@

ARG CONNECT_CLIENT_VERSION="0.8-rc"

Choose a reason for hiding this comment

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

future improvement: the latest CONNECT_CLIENT_VERSION could be extracted from the release information in the repo https://github.com/seqeralabs/studio-templates. As we are still finalizing versioning of the connect and cliejt templates, this is something that can be added later

Copy link

@endre-seqera endre-seqera left a comment

Choose a reason for hiding this comment

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

looks good!

@pinin4fjords
Copy link
Collaborator Author

Thanks @endre-seqera !

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.

3 participants