https://nf-co.re logo
Join Slack
Powered by
The Nextflow Training is moving to strict syntax, ...
# infrastructure
p
The Nextflow Training is moving to strict syntax, at the moment the only part that isn't updated is the nf-core training. This uses two pipelines: the #C071K9XPF47 pipeline and one that the users create from #CE5LG7WMB I think we should update #C071K9XPF47 ASAP. I'm going to try to take a stab at that now. If folks are around to help / review etc. that'd help a lot. It's going to be a day of multitasking for me 😅
m
you're missing the re-generation of the snapshot
p
yeah working on that now
The pre-commit dependency missing for nf-core lint is more confusing though confused dog
👀 1
m
That's a fun one
p
@mashehu @Júlia Mir any ideas?
I can install it, but then that (ironically) breaks linting as the github actions workflow differs from the template
Not sure why this hasn't been happening to other pipelines confused travolta
Triggered when 
nf-core pipelines lint
is trying to run 
dump_json_with_prettier
and finds that 
pre-commit
is not installed on the runner
ok I'm going to install pre-commit and then skip linting the actions file to try to get tests to pass.
j
having a look now
p
I guess this is fixed on 
dev
becauase we changed how 
requirements.txt
works (we removed it)
ok I think I figured it out
I ran 
nf-core pipelines lint . --fix
locally but have the 
dev
version of tools
This updated some files (like the github actions workfllow) for dev, which was incompatible with the 3.5.1 reelase it's running on CIO
j
ok! then we will need the tools release for this to work
p
nah it's ok
I need to run pipelines lint fix locally with stable, instead of dev
I think that should solve it
I thought that the tools release wasn't going to have any template changes on it though?
ok I think I'm getting there now
🙏 1
m
shout if you need anything
p
just pushed a commit. Hopefully that'll do it..
Had to resort to fixing stuff by hand, no LLM or 
--fix
or anything. Ugh! 😆
🤣 1
m
Good old days
p
except it didn't work
ok 3rd time lucky
yasss
I think that #C071K9XPF47 is now ready for release - PR that bumps versions and updates to meet strict syntax requirements here: https://github.com/nf-core/demo/pull/50
2 Views
Open in Slack
PreviousNext
Powered by