ICE 28189

[emoon]

No description provided.

[steveklabnik]

Ah ha! our first success! Travis failed because this has been fixed in the meantime.

So that you get credit, I'm going to merge this anyway, and then remove it in another commit. Thanks so much! 🎊 🎉 💯

[emoon]

Ah!

I was wondering why it was actually failing travis but I got the ICE locally but then it makes sense :)

[ghost]

and then remove it in another commit

But wait aren't you going to keep it in a separate category called regressions(which are supposed to never fail) ? What if it comes back? :)

[steveklabnik]

That's why the tracking bug is moved to E-needs test. That is what should keep it from coming back, the bug isn't fixed until there's a regression test in place.

Though a category might be fun...

[ghost]

Got it.
But I mean, you could make another dir into which you just move the ones that pass, and test.sh will (also) fail if any of those fail, this way work is not lost is commit history. Or something like that.

EDIT: or, I don't know how travis works, maybe you can have two test.sh files, one for regressions.