-
Notifications
You must be signed in to change notification settings - Fork 684
Print process dictionary in logs (#5940) #5944
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
base: master
Are you sure you want to change the base?
Print process dictionary in logs (#5940) #5944
Conversation
✅ Deploy Preview for nextflow-docs-staging canceled.
|
Oops.. I thought I'd been very cautious not to alter any existing behavior when modifying the code.. but the unit tests seem broken. After a quick look, I think one issue is that in Unfortunately, I'm off for two weeks so I can't have a look at why the tests are failing at the moment. (Also, I'm not using a proper IDE, only building and running, so I probably need to setup a proper dev environment to debug these issue.) |
Signed-off-by: kdesnos <[email protected]>
Signed-off-by: kdesnos <[email protected]>
Signed-off-by: kdesnos <[email protected]>
Signed-off-by: kdesnos <[email protected]>
Signed-off-by: kdesnos <[email protected]>
d53b9ae
to
6372902
Compare
Signed-off-by: kdesnos <[email protected]>
6372902
to
0bf2324
Compare
The PR is ready to be merged. |
…cates. Duplicates may appear when applying a configuration to an identical process alias used in several workflows. Signed-off-by: kdesnos <[email protected]>
I noticed that the "resolved" name of the included process may be ambiguous, in case the same alias is used in different workflow, for different original process. This is because when an include occurs in a workflow I'll fix this issue before reopening the PR. |
…esolved name is no longer kept, only the name with full workflow path is, as in reports. Signed-off-by: kdesnos <[email protected]>
…able Signed-off-by: kdesnos <[email protected]>
What
Update the logs in the following manner:
Replace workflow process name list, which mixes defined process names and aliased process names
with two lists: one of process definitions from each file, one of aliases used for each process.
Complement the following log line:
with typed list of inputs and outputs of the process.
Motivations