Fwd: [PEAR-BUG] Bug #19051 [Fbk->WFx]: remove error_reporting (for PEAR QA team)
[email protected] (Daniel Convissor) Mon, 21 Nov 2011 17:12:54 -0500
| Newsgroups | php.pear.qa |
|---|---|
| Message-ID | <[email protected]> |
Hi PEAR QA Team: What's your take on this, please. Thanks, --Dan ----- Forwarded message from PEARBug Database <[email protected]> ----- ID: 19051 Updated by: [email protected] Reported By: danielc at analysisandsolutions dot com Summary: remove error_reporting (for PEAR QA team) -Status: Feedback +Status: Wont fix Type: Bug Package: PHP_CodeSniffer Package Version: SVN PHP Version: 5.4.0alpha1 -Assigned To: +Assigned To: squiz Roadmap Versions: New Comment: -Status: Feedback +Status: Wont fix -Assigned To: +Assigned To: squiz PHP_CodeSniffer is not a library. It's an application and it requires it's own error reporting settings so it doesn't have to conform to the same error reporting settings users may have set in php.ini for their other PHP projects. I'm closing this issue because I don't see any reason to remove error reporting and potentially have errors hidden from users, meaning less or inaccurate bug reports. Previous Comments: ------------------------------------------------------------------------ [2011-11-21 14:22:11] danielc Library scripts should not set error reporting at all. Decisions on error reporting are up to the users of the library. ------------------------------------------------------------------------ [2011-11-21 06:09:16] squiz -Status: Open +Status: Feedback Sorry, but why exactly am I removing error_reporting settings from files that have nothing to do with unit testing? Was this an automated message? ------------------------------------------------------------------------ [2011-11-16 04:04:18] danielc Added #patch bug:19051;patch:qa;revision:1321416258;. ... snip ... -- T H E A N A L Y S I S A N D S O L U T I O N S C O M P A N Y data intensive web and database programming http://www.AnalysisAndSolutions.com/ 4015 7th Ave #4, Brooklyn NY 11232 v: 718-854-0335