-
Notifications
You must be signed in to change notification settings - Fork 634
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add/process.dockerfile #3916
Add/process.dockerfile #3916
Conversation
Signed-off-by: Marco De La Pierre <[email protected]>
Signed-off-by: Marco De La Pierre <[email protected]>
Signed-off-by: Marco De La Pierre <[email protected]>
Not super sure about this, need to think a bit |
I knew it, sure, also happy to discuss here or email or slack |
No rush at all |
I have just got the github app on the phone, I can be more responsive now 😉 |
Last one before I board 8-) I will write some more context about this PR over the next few days, as there are a couple of points I wanted to make...but today it was either the PR or the plane! |
I've commented about this here |
81f7cb7
to
8a43489
Compare
I'm bit concerned about overloading Wave with too many features, and I haven't seen any real user demand for this specific capability. I suggest closing this and reconsidering in the future if needed. |
I agree - the community has spoken, so let's go with the K.I.S.S. approach :-) |
@pditommaso have fun!
this is a small one.
it comes out of some of our discussions within nextflow-io/nf-wave#4
have the loooong flight in a few hours .. touching base in Milan tomorrow!