Wildcard warning fix

[jboero]

No longer suggests using wildcards when you've used wildcards.

[pep8speaks]

Hello @jboero! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

• In the file dnf/cli/cli.py:

Line 606:17: E128 continuation line under-indented for visual indent
Line 607:17: E128 continuation line under-indented for visual indent

Comment last updated at 2024-03-27 11:42:25 UTC

[jboero]

Hi @pep8speaks not sure if you're a bot or human but the code works fine for me. The not '*' in text is valid.

[m-blaha]

Yeah, it works. But according to PEP 8 the syntax should be '*' not in args[0].

[MarkBenjamin]

A suggestion:

        if not matches:
            if args[0].startswith("*/"):
                raise dnf.exceptions.Error(_('No matches found.'))
            raise dnf.exceptions.Error(_('No matches found. If searching for a file, '
                                         'try specifying the full path or using a '
                                         'wildcard prefix ("*/") at the beginning.'))

[jboero]

Wildcards don't technically need to be at the front though. Hmm let me try the original suggestion first.

[jboero]

These failures look like false negatives to me now. The behaviour affects test expectations.

[jan-kolarik]

@jboero Hi, seems like this one has been a bit forgotten. Could you please try rebasing it so the tests are rerun? Additionally, the string provided to the print function should be wrapped using the _ operator to make it translatable.

[jboero]

Hi @jan-kolarik thanks for the routine check. This one apparently isn't worth merging so will skip for now.