Tim Rogers

@tubaxenor Thanks for this PR, and sorry that it has been very slow to get a review. Have you tried this code yourself? I'm surprised that you need to take...

Are we sure this is a problem? The Capybara readme says the following: Capybara's RSpec matchers, however, are smart enough to handle either form. The two following statements are functionally...

I'm not going to have time to jump on this for now I'm afraid!

@dylan-smith Assigning this to you since you've been working on it - but no pressure on your to keep going! Feel free to drop/hand over if you want 👼

This is a fantastic issue, because we should definitely do what we can to validate input upfront and return great error messages. But it'll benefit from being broken down into...

> would this handle the case where you have your repo in AzDO which has no upper file size limit (and as far as I can tell no LFS setup)...

@dylan-smith Did this get done in the end? 👼

I'm not sure the description above is correct, as the `GrantMigratorRole` mutation seems to only support an organization as a target. But we could nevertheless implement this!

I do definitely think we should do this. It would be easy to add in the `migrate-repo` command at least. I guess it's likely to be harder in `generate-script`, as...

That makes sense. What do you mean by “generate scenario”? I’m not necessarily proposing that we work on this issue straight away - I just created it to track a...