Ryan A. Hagenson

@stefandd I have added myself as a reviewer to remind myself to look at it this evening. I think Sean has been busy lately so I want to ensure you...

@stefandd I have added some comments to continue the conversation. Biggest update for my thinking is, after thinking on it more, convincing myself a parameter rearrangment approach is a bad...

@stefandd Excellent to see the rewrite! I will review this when I am home from work today. I really like/support the changes you are putting forward so I hope the...

Sean, I presume your comment was due to my talking about the docstring for `Reverse`. I took the docstring to mean "`Reverse` is meant to be `Range` in reverse" I...

If we go with the changes here as written without any change to `Reverse` then `Range` and `Reverse` would no longer be related in the same way and the docstring...

Commenting so this get reviewed during Sync eventually. There are a handful of methods without docstrings, but I do not think it is clear from the current state of `String`...

I like that a lot more. I was trying to think of a way to get rid of the match because I did not like one condition being the fall...

@SeanTAllen I wanted to ping you on this one while I was looking for issues to highlight. As far as I can tell, this issue is "fixed" following improvements to...

This profile is a WIP as I am sure there are at least a handful of oversights or better ways to configure usage. The corresponding documentation for the cluster itself...

Something is definitely buggy about this config as-is. I attempted to run nf-core/sarek with it (and the test data) and it fails with a stack overflow for `NFCORE_SAREK:SAREK:PREPARE_INTERVALS:CREATE_INTERVALS_BED` -- removing...