On Friday, December 7, 2018 10:19:45 AM CST Rasmus Schultz wrote: > I noticed the introduction of two PHPUnit-specific test-dependencies into > the psr/log package: > > https://github.com/php-fig/log/tree/1.1.0/Psr/Log/Test > > How is this in any way acceptable? > > The dependency isn't even declared in the "composer.json" file - but that's > kind of besides the point. > > First of all, PHPUnit isn't PHP - introducing facilities for a specific > test-framework is extremely opinionated and does not belong in an package > with interfaces designed to bridge projects and provide interoperability > between different frameworks. (PHPUnit being a test-framework.) > > Moreover, these facilities aren't a dependency, at all - you can use this > package just fine without it. There is no rational reason not to package > these dependencies separately. > > These facilities belong in a different package that depends on psr/log, so > you can version them separately. Stuffing these into the package itself is > extremely controversial and could create a serious problem with versioning. > (having to version the interfaces as 2.0 for a breaking change to the > test-facilities!) > > I'm calling for a bugfix release to remove these ASAP. > > Who's even responsible for this? > > There's no issue-tracker on the GitHub project, and I don't see any > discussion about it in this forum. > > wtf?
One of them has been there for years. The other was just added recently with little notice. This has been discussed before, and frankly I agree that the tests should not be in the package at all. We have -util packages for this sort of thing. Related discussion: https://github.com/php-fig/log/pull/52#issuecomment-378323725 --Larry Garfield -- You received this message because you are subscribed to the Google Groups "PHP Framework Interoperability Group" group. To unsubscribe from this group and stop receiving emails from it, send an email to php-fig+unsubscr...@googlegroups.com. To post to this group, send email to php-fig@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/php-fig/1731374.OImHZf50bf%40vulcan. For more options, visit https://groups.google.com/d/optout.
signature.asc
Description: This is a digitally signed message part.
