#general
- e
enough-analyst-54434
09/02/2015, 11:07 PMWhich your change ~yesterday woudl have fixed - e
enough-analyst-54434
09/02/2015, 11:07 PMgotcha - b
bored-art-40741
09/02/2015, 11:07 PMThis is a ticking time bomb for us, and it also prevents us from using goodies like name hashing - e
enough-analyst-54434
09/02/2015, 11:08 PMOpen source the resolver! - b
bored-art-40741
09/02/2015, 11:08 PMIt's code dumped - e
enough-analyst-54434
09/02/2015, 11:08 PMWe can all look at it, unit test and fix - b
bored-art-40741
09/02/2015, 11:08 PMBut not entirely appropriate for contrib - e
enough-analyst-54434
09/02/2015, 11:08 PMcode dumps really don;t count - b
bored-art-40741
09/02/2015, 11:08 PMI know - e
enough-analyst-54434
09/02/2015, 11:08 PMyou can;t do that and expect action - b
bored-art-40741
09/02/2015, 11:08 PMAnd I don't have the time to generalize it - e
enough-analyst-54434
09/02/2015, 11:08 PMk - b
bored-art-40741
09/02/2015, 11:08 PMI don't expect action on it - b
bored-art-40741
09/02/2015, 11:08 PMAt least not for a month or more while I take care of other things and Mateo spins up internally - b
bored-art-40741
09/02/2015, 11:09 PMWhat I'm getting at here is that this option is a temporary hack - b
bored-art-40741
09/02/2015, 11:09 PMIt gives us an escape hatch that relieves pressure, and it goes away when we move off the global strategy - b
bored-art-40741
09/02/2015, 11:09 PMIt's a simple, blunt instrument that's extremely useful to us and completely unintrusive in terms of code - b
bored-art-40741
09/02/2015, 11:10 PMAnd my second attempt to get this to be testable just failed - b
bored-art-40741
09/02/2015, 11:11 PMWhich isn't unexpected, since pants has hardened itself against the type of failure I'm trying to generate in all possible ways - b
bored-art-40741
09/02/2015, 11:11 PMI'm hoping I can get away with this option being untested short of demonstration that it doesn't break anything and works for us internally - b
bored-art-40741
09/02/2015, 11:11 PMWhere we have the only repro - b
bored-art-40741
09/02/2015, 11:12 PMThough really - b
bored-art-40741
09/02/2015, 11:12 PMI can just patch and fork - b
bored-art-40741
09/02/2015, 11:12 PMSince it's hopefully temporary - e
enough-analyst-54434
09/02/2015, 11:12 PMsg - e
enough-analyst-54434
09/02/2015, 11:13 PMI missed the day we crossed, but woohoo all you test writers!: https://coveralls.io/r/pantsbuild/pants - e
enough-analyst-54434
09/02/2015, 11:13 PM75%metal 1 - e
enough-analyst-54434
09/02/2015, 11:14 PMI've never been on a big project with better than 85% and we're not far off for some definition of far off - b
bored-art-40741
09/02/2015, 11:15 PMWe also undercount when considering integration tests, I think - e
enough-analyst-54434
09/02/2015, 11:15 PMyup - those contribute 0%