I like you are adding a lot of linters, and fix the issues.

Maybe you could have split the PR in two:

• one for the linters, and their fixes
• one for the migration tests

It's OK, but it's a bit surprising to see them mixed. I mean it's uneasy to review the code related to the migration tests

There are multiple commits, but here I'm afraid with so many changes, people might not review these specific changes.

You know the meme:

• 50 lines changed: 100 comments
• 5000 lines changed: LGTM 👍

BTW, I'll try to see if there are other linters you could enable

I like you are adding a lot of linters, and fix the issues.

Maybe you could have split the PR in two:

• one for the linters, and their fixes
• one for the migration tests

It's OK, but it's a bit surprising to see them mixed. I mean it's uneasy to review the code related to the migration tests

There are multiple commits, but here I'm afraid with so many changes, people might not review these specific changes.

You know the meme:

• 50 lines changed: 100 comments
• 5000 lines changed: LGTM 👍

BTW, I'll try to see if there are other linters you could enable

Yeah, I agree. A split could've definitely been better. A small change snowballed into a pretty major refactor.
I'll keep that in mind for the future 👍

And, yeah, the meme is definitely spot on :D

Thanks for the reviews, @ccoVeille!