Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

updated output format for reports #1191

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

i582
Copy link
Contributor

@i582 i582 commented Aug 18, 2022

Before

MAYBE   implicitModifiers: Specify the access modifier for \B::f method explicitly at /Users/petrmakhnev/noverify/script.php:106
    function f(): parent {
             ^
<critical> WARNING unused: Variable $a is unused (use $_ to ignore this inspection or specify --unused-var-regex flag) at /Users/petrmakhnev/noverify/script.php:107
        $a = 100;
        ^^

After

MAYBE   implicitModifiers: Specify the access modifier for \B::f method explicitly at /Users/petrmakhnev/noverify/script.php:106
    |
106 |    function f(): parent {
    |             ^

<critical> WARNING unused: Variable $a is unused at /Users/petrmakhnev/noverify/script.php:107
    |
107 |        $a = 100;
    |        ^^ use `$_` to ignore (see --unused-var-regex flag)

@i582 i582 added the enhancement New feature or request label Aug 18, 2022
@i582 i582 added this to the next milestone Aug 18, 2022
@i582 i582 requested a review from Danil42Russia August 18, 2022 13:21
@Danil42Russia
Copy link
Contributor

This seems to break the output in AdmStrom

@i582 i582 removed their assignment May 5, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants