Skip to content

fix: issue with dirs relative to .test#8

Merged
m-jahn merged 1 commit into
mainfrom
dev
Jul 13, 2026
Merged

fix: issue with dirs relative to .test#8
m-jahn merged 1 commit into
mainfrom
dev

Conversation

@m-jahn

@m-jahn m-jahn commented Jul 13, 2026

Copy link
Copy Markdown
Member

closes #7

Copilot AI review requested due to automatic review settings July 13, 2026 14:14
@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

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adjusts the Apptainer build workflow to work when the Snakemake containerization recipe is generated under .test, addressing broken %files paths reported in issue #7.

Changes:

  • Run the Apptainer build step from the .test working directory.
  • Update the apptainer build invocation to use paths relative to .test.

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

run: |
sudo apt-get install -y uidmap
apptainer build --fakeroot apptainer.sif apptainer.def
apptainer build --fakeroot ../apptainer.sif ../apptainer.def

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

no, the output from containerize is ./apptainer.def, in root dir.

@m-jahn
m-jahn merged commit 2a2fa6d into main Jul 13, 2026
3 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.

Local envs not available if Apptainer recipe is created on .test

2 participants