Skip to content

Commit

Permalink
Merge pull request #365 from nicwortel/remove-leading-blank-line-from…
Browse files Browse the repository at this point in the history
…-text-renderer

Remove the leading blank line from the TextRenderer
  • Loading branch information
ravage84 committed Apr 28, 2016
2 parents 2b9c241 + ca59154 commit 0995c85
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 3 deletions.
1 change: 0 additions & 1 deletion src/main/php/PHPMD/Renderer/TextRenderer.php
Expand Up @@ -65,7 +65,6 @@ class TextRenderer extends AbstractRenderer
public function renderReport(Report $report)
{
$writer = $this->getWriter();
$writer->write(PHP_EOL);

foreach ($report->getRuleViolations() as $violation) {
$writer->write($violation->getFileName());
Expand Down
2 changes: 0 additions & 2 deletions src/test/php/PHPMD/Renderer/TextRendererTest.php
Expand Up @@ -91,7 +91,6 @@ public function testRendererCreatesExpectedNumberOfTextEntries()
$renderer->end();

$this->assertEquals(
PHP_EOL .
"/bar.php:1\tTest description" . PHP_EOL .
"/foo.php:2\tTest description" . PHP_EOL .
"/foo.php:3\tTest description" . PHP_EOL,
Expand Down Expand Up @@ -131,7 +130,6 @@ public function testRendererAddsProcessingErrorsToTextReport()
$renderer->end();

$this->assertEquals(
PHP_EOL .
"/tmp/foo.php\t-\tFailed for file \"/tmp/foo.php\"." . PHP_EOL .
"/tmp/bar.php\t-\tFailed for file \"/tmp/bar.php\"." . PHP_EOL .
"/tmp/baz.php\t-\tFailed for file \"/tmp/baz.php\"." . PHP_EOL,
Expand Down

0 comments on commit 0995c85

Please sign in to comment.