Closed edmundmiller closed 6 months ago
nf-core lint
overall result: Passed :white_check_mark: :warning:Posted for pipeline commit 4c299ae
+| ✅ 185 tests passed |+
!| ❗ 4 tests had warnings |!
@edmundmiller I personally don't mind the change! Would you prefer to get feedback from someone with more experience/knowledge before merging though?
Thanks! I think we just merge this, and we can go back to 4 spaces if it causes issues.
Actually, I don't know if this will mess with the template updates 🤔 I think we should go with 4 spaces.
I have a strong feeling it would mess with template updates haha So going back to 4 spaces now?
Switched to 4 spaces 👍🏻
Main thing was setting
Which was the style that I found. Not sure how others feel about deviating from the rest of nf-core on the Nextflow style specifically, but I think getting it all in one style first is a good step!