Better Ansi Output

[Kazuto]

Description

We've just started using phpmd in an already established codebase and the ANSI output was just confusing to read or to understand what belongs to what.

It's unclear for users who took the first look at the output that it's divided like

File:
-----
Findings

But instead of being read as

-----
Findings
File:
-----

Adding a blank like certainly would make the output more readable and easier on the eyes, like this:

Before	After

I have not looked into the codebase to say if I could provide a PR.

Checks before submitting

• Be sure that there isn't already an issue about this. See: Issues list
• Be sure that there isn't already a pull request about this. See: Pull requests
• Tell if you have the option to provide the code for this proposal.
• This issue is about 1 feature proposal and nothing more.
• The issue has a descriptive title. For example: "Add JSON render option".

[kylekatarnls]

Sounds fair to me.

[himanshu007-creator]

Hi, i would like to work on this issue. Please point me the file. Thanks

[tvbeek]

De different renders are place in src/main/php/PHPMD/Renderer so the Ansi renderer is: https://github.com/phpmd/phpmd/blob/master/src/main/php/PHPMD/Renderer/AnsiRenderer.php

To test for that renders is placed in: https://github.com/phpmd/phpmd/blob/master/src/test/php/PHPMD/Renderer/AnsiRendererTest.php

Because you say that you want to work on this issue I assign it to you so that it is clear for everyone. (And thanks that you want to help 👍 )

Please say it if you want to have more information 😃

[himanshu007-creator]

Can you tell where exactly to insert the new line? I am new to PHP

[kylekatarnls]

This line writes the file header to the output:
https://github.com/phpmd/phpmd/blob/master/src/main/php/PHPMD/Renderer/AnsiRenderer.php#L41

[kylekatarnls]

Hello @Kazuto, thanks to @himanshu007-creator there is now one more new line in between, you can test it requiring "phpmd/phpmd": "dev-master"