GivenAndUsing rule duplicates using keyword when explicitly passing a using param

[mrdziuban]

Thank you for these rules, it makes the migration to new Scala 3 syntax a lot simpler!

I've found that the GivenAndUsing rule can produce invalid syntax when existing code is already passing a using param explicitly.

For example given this code:

def test(using i: Int): Int = i

test(using 1)

The GivenAndUsing rule rewrites it to this:

def test(using i: Int): Int = i

// Note duplicated `using` keyword here
test(using using 1)

I'm guessing something around here needs to check whether there's already a using keyword?

[ingarabr]

It's awesome that you found them useful and I'm happy that they're in use :-)

It looks like you're on to something where we need to add a guard against being a using (something like _.is[Mod.Using]). Could you look into it and create a PR for it?

[mrdziuban]

Could you look into it and create a PR for it?

Sure thing, just opened #31