SWAP flake8-gettext (INT)

[p1-dta]

Summary

Two rules documentation seems to be swapped:

• INT002: Checks for str.format calls in gettext function calls.
  
• INT003: Checks for printf-style formatted strings in gettext function calls.

Expected detection:

• INT002: _("Hello, {}!".format(name)) # Looks for "Hello, Maria!".
• INT003: _("Hello, %s!" % name) # Looks for "Hello, Maria!".

Actual documentation:

• INT002: _("Hello, %s!" % name) # Looks for "Hello, Maria!".
• INT003: _("Hello, {}!".format(name)) # Looks for "Hello, Maria!".

I wonder if this is intended.

I didn't checked if the rule INT002 actually detect str.format or printf style, but in any case, there is an inconsistency somewhere.

[ntBre]

Nice catch! If I'm parsing the results on the playground right, it looks like the examples in the documentation are swapped and the implementations are correct.

[MichaReiser]

@p1-dta are you interested in PRing a fix or @ntBre, are you already working on it?

[ntBre]

I haven't started working on it. @p1-dta feel free to go ahead if you're interested, or we can put the help wanted label on.

I am happy to work on it if nobody else does, though.

[MichaReiser]

I think this is bad enough that we should fix it ourselves, unless @p1-dta is interested

[ntBre]

I'll keep it in my inbox and check back later today.

[p1-dta]

Hi, I'm back, sorry for being late, I don't use GH on a daily basis, so I missed the notification.
I not looking for PRing if someone else have the time to tackle it.
Thanks for fixing it.