Closed pbelmann closed 4 weeks ago
Nextflow fixownership issue does not work in all containers, which is due to the fact that chown --from is not possible in busybox docker images.
This PR fixes the file pemissions of output files manually.
Thank you for submitting this PR.
Before merge:
The PR must be reviewed by one of the team members.
Please check if anything in the Readme must be adjusted, or added (development-setup, production-setup, user-guide).
PRs with new modules or workflow interfaces must include tests according to the developer guidelines.
The new code is readable, well commented and should adhere to our developer guidelines.
Before merging it must be checked if a squash of commits is required.
Nextflow fixownership issue does not work in all containers, which is due to the fact that chown --from is not possible in busybox docker images.
This PR fixes the file pemissions of output files manually.
PR review guidelines
Thank you for submitting this PR.
Before merge:
The PR must be reviewed by one of the team members.
Please check if anything in the Readme must be adjusted, or added (development-setup, production-setup, user-guide).
PRs with new modules or workflow interfaces must include tests according to the developer guidelines.
The new code is readable, well commented and should adhere to our developer guidelines.
Before merging it must be checked if a squash of commits is required.