New GitHub actions from the Nextflow training repo to run

nextflow lint

and

nextflow lint -format

and post results as a comment to the PR, never failing the CI: • Run lint • Post as comment • Example comment This is intended as an early-stage prod for migrating to strict syntax and auto-formatting. Once conversion is complete, it can be ditched and replaced with simpler

pre-commit

actions.

https://github.com/nextflow-io/training/blob/master/.github/workflows/nextflow-lint.yml#L123 I wouldn't use <sup> for longer text. it's too tiny

I found it too attention grabbing otherwise, I like the <sup> 🙄

why not <small>?

but looks very nice otherwise (maybe ordering

fail

,

warn

,

pass

would make more sense than

fail

,

pass

,

warn

Where do you see this? I only see fail warn pass?

I think we already used pre-commit in #C08TKFDUNHG and other #C07PTBHLK7D, because of the smaller scope of the nextflow scripts there

I meant this. but it didn't mean

warn

more

--fix

i guess

Yeah, it's a different class of thing - files can have no errors and be changed by auto-formatting

isn't that basically a warning?

Depends on your view, but kinda?