nextflow icon indicating copy to clipboard operation
nextflow copied to clipboard

Fix `process.ext` set with `withName` overrides all processes

Open jorgeaguileraseqera opened this issue 2 years ago • 5 comments

We maintain in profileStack a list of sections visited but we were checking if the property requested was only into the scope of the current stack instead to check if present in the last profile

Another issue was we were checking if the var is present with if( current.scope[name] ) but this expression creates the variable if not present, so I've changed for containsKey

Lastly a little refactor removing the if-else-if chain for independents if who check if previous didn't find a valid result to return

closes #2422

jorgeaguileraseqera avatar Apr 11 '22 06:04 jorgeaguileraseqera

The guru of ConfigSlurper! 😄

pditommaso avatar Apr 11 '22 08:04 pditommaso

Can you give please have a look at the conflict?

pditommaso avatar Jun 12 '22 18:06 pditommaso

done

after some errors in the pipeline due GitHub cache issue at the end, the pipeline ends ok

jorgeaguileraseqera avatar Jun 14 '22 07:06 jorgeaguileraseqera

Unknown status. Closing it

pditommaso avatar Jul 05 '23 09:07 pditommaso

This PR needs to be cleaned up but I believe it can be salvaged. Reopening

bentsherman avatar Nov 08 '23 15:11 bentsherman