docs: Add guide for bin scripts #933
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
gwright99
requested changes
Nov 21, 2025
Member
gwright99
left a comment
gwright99
left a comment
There was a problem hiding this comment.
Left comments and thoughts. Good first pass but I feel that elaboration could be helpful for the average reader who doesn't yet have a full grasp on the inner workings of these various ecosystem components.
Tidying up PR - resolving agreed suggestions
…s/wave into docs-guide-bin-scripts
gwright99
approved these changes
Dec 8, 2025
Member
gwright99
left a comment
gwright99
left a comment
There was a problem hiding this comment.
Approving to get PR unblocked and on understanding discussed items will be handled in separate document at some future point.
Member
Author
pditommaso
reviewed
Dec 11, 2025
|
|
||
| When you enable Fusion or use the AWS Fargate executor, Wave bundles scripts from the project `bin/` directory into a container layer. This provides better performance but ties scripts to specific execution environments. Scripts are added to `/usr/local/bin/` in the container image, making them directly executable without path qualification. Remote `bin/` directory upload is disabled. | ||
|
|
||
| Modifying scripts in the project `bin/` directory modify the container fingerprint for all Wave containers in the workflow. See [Wave container fingerprinting](#wave-container-fingerprinting) for more information. |
Collaborator
There was a problem hiding this comment.
This is a bit too technical, i'd focus it will result in creating a modified container, and therefore invalidating the cache for previous runs
pditommaso
reviewed
Dec 11, 2025
Co-authored-by: Paolo Di Tommaso <paolo.ditommaso@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR adds documentation for Wave's
binscript bundling behavior in Nextflow pipelines. This guide helps users understand which scripts get packaged into container images and how this affects container fingerprinting and caching.This PR is intentionally limited in scope to address a specific user pain point: understanding how Wave handles bin scripts from different directories (
${projectDir}/bin/vs.${moduleDir}/resources/usr/bin/) and when those scripts are bundled into container layers.The guide covers:
binscripts are bundledI've tried to keep the scope limited and focus on the bin behavior without going too deep. Not covered here: