Juliette

Results 2034 comments of Juliette

Update: I've now gone ahead and implemented option 2. I'll be pulling the related changes over the next few days. The reasons to implement option 2 (rather than @fredden's option...

PR #647 is now open if anyone wants to test the implementation before I merge it (will leave it open for ~week).

Considering I introduced this (even though, I did it when I was much less experienced) and that I can still come up with a reason why it may be good...

Given the opinions on this, let's remove the support for const aliases. If/when the "is this a global constant" check moves to PHPCSUtils this may need another look, but that's...

I'm now in two minds... should this be allowed to go into the next patch release (and regarded as a bug fix) or does this need to go into the...

@thomas-price Could you explain **why** you think the rule should be relaxed in that case ? I mean, debug printing is still debug printing, whether printed to screen or returned...

@thomas-price I must still be missing something, but your use case sounds 100% like debugging, which should not be in production code. If you need to handle errors from 3rd...

Nor me ;-) Having said that, I can see builds reporting back again over the last few hours. The problem with them not reporting back, is, of course, that if...

This issue is still happening, but intermittently/randomly. I can have three consecutive PRs running a build and two will get Coveralls reporting back, one will be stuck on "pending completion"...