Task #66437

Simplify PHP lint Travis execution

Added by Mathias Brodala over 4 years ago. Updated about 1 year ago.

Status:
Closed
Priority:
Should have
Category:
Code Cleanup
Target version:
Start date:
2015-04-15
Due date:
% Done:

100%

TYPO3 Version:
6.2
PHP Version:
Tags:
Complexity:
Sprint Focus:

Description

The execution of PHP lint within Travis is rather verbose ATM and requires a temporary file.

This should be simplified.

Associated revisions

Revision ccd227e5 (diff)
Added by Mathias Brodala over 4 years ago

[TASK] Simplify PHP lint execution

This simplifies PHP lint execution in Travis by ensuring lint
errors are outputted to stderr and drops everything else. If
any check fails, the exit code 1 is returned.

The Composer vendor directory is excluded from linting.

Resolves: #66437
Releases: master
Change-Id: I588a880f548c248a0db44f3a3f1274fcec2edd00
Reviewed-on: http://review.typo3.org/38704
Reviewed-by: Wouter Wolters <>
Tested-by: Wouter Wolters <>
Reviewed-by: Markus Klein <>
Reviewed-by: Christian Kuhn <>
Tested-by: Christian Kuhn <>

Revision b52b9ae7 (diff)
Added by Mathias Brodala over 4 years ago

[TASK] Simplify PHP lint execution

This simplifies PHP lint execution in Travis by ensuring lint
errors are outputted to stderr and drops everything else. If
any check fails, the exit code 1 is returned.

The Composer vendor directory is excluded from linting.

Resolves: #66437
Releases: 6.2
Change-Id: I588a880f548c248a0db44f3a3f1274fcec2edd00
Reviewed-on: http://review.typo3.org/39425
Reviewed-by: Andreas Fernandez <>
Tested-by: Andreas Fernandez <>
Reviewed-by: Wouter Wolters <>
Reviewed-by: Christian Kuhn <>
Tested-by: Christian Kuhn <>

History

#1 Updated by Gerrit Code Review over 4 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/38704

#2 Updated by Gerrit Code Review over 4 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/38704

#3 Updated by Mathias Brodala over 4 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

#4 Updated by Gerrit Code Review over 4 years ago

  • Status changed from Resolved to Under Review

Patch set 1 for branch TYPO3_6-2 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/39425

#5 Updated by Mathias Brodala over 4 years ago

  • Status changed from Under Review to Resolved

#6 Updated by Benni Mack about 1 year ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF