JulianFlesch
JulianFlesch
Ready for review, if you have the time @mashehu @mahesh-panchal ! 🎉 ## Summary of latest changes: * A way to retry builds for the nf-Core/tools container was added due...
@mirpedrol > The new template file `.devcontainer/setup.sh` should be skipped by a template feature. We should remove the current `gitpod` feature and update the `codespaces` feature to add the new...
Can you especially have a look at my changes to `template_features.yaml` (see [d3a465b](https://github.com/nf-core/tools/pull/3569/commits/d3a465b9f1ec5ccb6e6a4b90f2cadb9b157ce84e)) if the new files need to be added anywhere else @mirpedrol ? Also to fix the tests...
Small comment regarding [3008aae](https://github.com/nf-core/tools/pull/3569/commits/3008aaefe4af8e7c3e62cedd4fb64aef46f41616): I removed the use of `$localWorkspaceFolder` (which was introduced to make docker with mounts from the host possible and easy), because it is not working in...
Welcome back and thanks for having a look @mahesh-panchal This PR changed quite a bit, but this update still sums it up well: https://github.com/nf-core/tools/pull/3569#issuecomment-3103466521 The main things I could use...
### Final changes to address the size of the resulting devcontainer **Problem 1:** The features added a lot of size (roughly 300MB each. >1GB for the local nextflow feature!) **Problem...
@mahesh-panchal The error is because the image is not yet pushed to Dockerhub. This will work once the first build action (after a pull to main) is run. For now,...
> To test on the nf-core modules repo, which files are the important ones I should transfer to a branch? The files `.devcontainer/devcontainer.json` and `.devcontainer/setup.sh` I would assume. Some testing...
* Test still missing ( @ningyuxin1999 ) * Linting still missing * Sorting the resulting `meta.yml` after running create() still missing * Refactoring ModuleInfo to be helpful here -> TODO:...
Hi @muffato thanks for your comments! There is currently `["amend", "copy", "remote"]` > The proposed implementation/purpose of -u amend is to create a copy of the pipeline that entirely relies...