Skip to content

feat: added pixi env, test tasks, and updated apptainer workflow to handle new --containerize args#4

Merged
m-jahn merged 4 commits intomainfrom
dev
Apr 24, 2026
Merged

feat: added pixi env, test tasks, and updated apptainer workflow to handle new --containerize args#4
m-jahn merged 4 commits intomainfrom
dev

Conversation

@m-jahn
Copy link
Copy Markdown
Member

@m-jahn m-jahn commented Apr 24, 2026

No description provided.

Copilot AI review requested due to automatic review settings April 24, 2026 08:58
@github-actions
Copy link
Copy Markdown

Super-linter summary

Language Validation result
YAML_PRETTIER Pass ✅

All files and directories linted successfully

For more information, see the GitHub Actions workflow run

Powered by Super-linter

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a Pixi-managed environment and a small Snakemake-based test fixture to validate workflow execution and Apptainer containerization, and updates the Apptainer deployment workflow to use the new --containerize argument handling.

Changes:

  • Add pixi.toml + pixi.lock with tasks to run test workflow/containerization and build an Apptainer image.
  • Add a minimal .test/ Snakemake project (Snakefile, env YAML, sample FASTQ) to exercise conda + wrapper execution.
  • Update .github/workflows/deploy-apptainer.yml to generate an Apptainer recipe via a GitHub Action input (containerize-args) instead of Dockerfile→def conversion.

Reviewed changes

Copilot reviewed 6 out of 8 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
pixi.toml Defines the Pixi workspace, dependencies, and local tasks for running Snakemake tests/containerization.
pixi.lock Pins the Pixi environment for reproducible CI/local runs.
.test/Snakefile Minimal test workflow including conda usage and a wrapper invocation.
.test/test_env.yaml Minimal conda env used by the test rule.
.test/input.fastq Sample input used by the wrapper test rule.
.gitignore Ignores Snakemake/Pixi artifacts produced by local/CI runs.
.github/workflows/deploy-apptainer.yml Updates the deployment workflow to rely on containerize args support.
.gitattributes Marks pixi.lock as generated / non-diff / binary-merge to reduce noise and merge conflicts.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/deploy-apptainer.yml Outdated
Comment thread .github/workflows/deploy-apptainer.yml
Comment thread pixi.toml Outdated
@github-actions
Copy link
Copy Markdown

Super-linter summary

Language Validation result
YAML_PRETTIER Pass ✅

All files and directories linted successfully

For more information, see the GitHub Actions workflow run

Powered by Super-linter

@m-jahn
Copy link
Copy Markdown
Member Author

m-jahn commented Apr 24, 2026

Note: Do not merge this PR until the upstream dependency www.github.com/snakemake/snakemake-github-action is fixed. After that, this section in deploy-apptainer.yaml must be updated.

      - name: create apptainer recipe
        uses: m-jahn/snakemake-github-action@fix_apptainer
        with:

@github-actions
Copy link
Copy Markdown

Super-linter summary

Language Validation result
YAML_PRETTIER Pass ✅

All files and directories linted successfully

For more information, see the GitHub Actions workflow run

Powered by Super-linter

@m-jahn m-jahn merged commit 82b9578 into main Apr 24, 2026
2 checks passed
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