#general
- r
rough-minister-58256
08/12/2015, 5:14 AMvery cool - l
limited-country-20626
08/12/2015, 6:21 AM😀 - w
witty-crayon-22786
08/12/2015, 3:27 PMcongratulations! - m
mammoth-pharmacist-54215
08/12/2015, 5:02 PMI’m trying to update ci.sh so that python style doesn’t blow up ci by disabling the style checks in ci. The problem I’m running into is that some of our integration tests run pants directly and fail as well. For example: https://travis-ci.org/pantsbuild/pants/jobs/75298616 - m
mammoth-pharmacist-54215
08/12/2015, 5:03 PMShould I fix these style nits, set the default on the command line to false (this is how it currently exists in RB), or update the integration tests to disable style (this seems messy). If anyone has other suggestions on how to resolve this I’d be glad to hear them. - e
enough-analyst-54434
08/12/2015, 6:56 PMDisable in pants.ini is the only real option. Folks should be able to run ci.sh on their workstations too. So until the RB that kills all style errors goes in, style checks should be off. - e
enough-analyst-54434
08/12/2015, 6:56 PMpants.ini is also used by integration tests - so this should work globally - no CLI flags needed - m
mammoth-pharmacist-54215
08/12/2015, 6:57 PM@enough-analyst-54434 thanks, sounds good to me. - a
ancient-stone-50795
08/12/2015, 6:58 PM@limited-country-20626: have you found the hidden camera yet? 🎥 🛰️ 📺 - l
limited-country-20626
08/12/2015, 7:02 PMJohn said he was looking at me...now I know! - a
adventurous-twilight-44110
08/12/2015, 7:17 PMdoes anyone know if it’s possible to have CI run only a single job? - r
rough-minister-58256
08/12/2015, 7:31 PMtemporarily comment out of all of the other tests in your branch maybe? there’s no real way to parameterize the travis run afaict since its PR based. - c
cool-thailand-74745
08/12/2015, 7:37 PM@enough-analyst-54434 if you're around plz take a look at https://github.com/pantsbuild/pex/pull/148 - c
cool-thailand-74745
08/12/2015, 7:37 PMalso anyone else - r
rough-minister-58256
08/12/2015, 7:50 PMis there an unspoken protocol for "ship-it-ing" a PR - r
rough-minister-58256
08/12/2015, 7:54 PMkinda surprised github doesnt have something like that already - b
bored-art-40741
08/12/2015, 7:55 PMLike when is it okay to click the button? - b
bored-art-40741
08/12/2015, 7:55 PMOr how to push the change? - w
witty-crayon-22786
08/12/2015, 7:56 PMpeople use a squirrel.🐿️ 1 - w
witty-crayon-22786
08/12/2015, 7:56 PMthat one. - b
bored-art-40741
08/12/2015, 7:56 PMWhat? - w
witty-crayon-22786
08/12/2015, 7:57 PM - e
enough-analyst-54434
08/12/2015, 8:25 PM@cool-thailand-74745: lgtm - r
rough-minister-58256
08/12/2015, 8:36 PMnice @ squirrel - c
cool-thailand-74745
08/12/2015, 9:05 PM@rough-minister-58256 @enough-analyst-54434 ty - c
cool-thailand-74745
08/12/2015, 9:16 PM - e
enough-analyst-54434
08/13/2015, 4:08 AMVery neat stuff @cool-thailand-74745 - a
ancient-stone-50795
08/13/2015, 4:34 PMFYI ./pants help-advanced is broken. I’m working on an RB - h
happy-kitchen-89482
08/13/2015, 4:35 PMhuh - h
happy-kitchen-89482
08/13/2015, 4:40 PMyikes