#general
- e
enough-analyst-54434
05/15/2016, 11:48 PMQuestion being, does anyone know the status of the
project?:API: public - a
ancient-stone-50795
05/16/2016, 1:42 AM@enough-analyst-54434: At Square we don't have the resources to continually refactor our plugins on every release. That is the purpose of the API:public project. Feel free to refactor for the 1.1 branch though. - e
enough-analyst-54434
05/16/2016, 1:43 AMThat is not my point - e
enough-analyst-54434
05/16/2016, 1:43 AMMy point is, stability is a good enough goal, but fwict we have made a half-hearted attempt at it - e
enough-analyst-54434
05/16/2016, 1:44 AMThe codebase is confusingly 1/2 (guessing) marked - e
enough-analyst-54434
05/16/2016, 1:44 AMWhich seems almost worst than no marking at all - a
ancient-stone-50795
05/16/2016, 1:44 AMI don't think that is at all fair. We spent a lot of time going through our plugins, culling out the crap that shouldn't be kept stable based on what folks are actually using and refactoring stuff to make more things private. - e
enough-analyst-54434
05/16/2016, 1:45 AMOK, I think I need to write a tool to point out the holes then - e
enough-analyst-54434
05/16/2016, 1:45 AMThere seem to be alot and this is all autamatable given roots - a
ancient-stone-50795
05/16/2016, 1:46 AMIf that makes things better then that's great. - e
enough-analyst-54434
05/16/2016, 1:47 AMYeah, I think the intent was good, but the result - with beginners eyes - is just confusing and inconsistent - e
enough-analyst-54434
05/16/2016, 1:47 AMI'll give it a whirl - a
ancient-stone-50795
05/16/2016, 10:15 AMI know that early on we struggled with how plugins need access to an awful lot of the pants internals to do useful work. - w
witty-crayon-22786
05/16/2016, 6:06 PMheads up: want to try and start a discussion of release policy in the #C18RRR4JK room - b
bored-art-40741
05/16/2016, 6:19 PMDoes anyone know the general status of the nodejs pants contrib module? - b
bored-art-40741
05/16/2016, 6:19 PMIs it used anywhere actively? - b
bored-art-40741
05/16/2016, 6:19 PMI'm interested in writing some light pants tooling around some react workflows we're developing internally - w
witty-crayon-22786
05/16/2016, 6:19 PM@bored-art-40741: it's seeing more and more usage internally - w
witty-crayon-22786
05/16/2016, 6:20 PMand it sounds like medium will be trying to pick it up soon - b
bored-art-40741
05/16/2016, 6:21 PMInteresting - w
witty-crayon-22786
05/16/2016, 6:23 PMour environment sortof forces people to use it though... we push back against installing anything in the CI environment, but since pants carries node 5.5.0 "along with it", it's the easiest way to get things running - w
witty-crayon-22786
05/16/2016, 6:23 PMi'm pretty sure folks aren't using it as part of their workflow locally though - b
bored-art-40741
05/16/2016, 6:23 PMYeah this is strictly for CI so far - b
bored-art-40741
05/16/2016, 6:23 PMAnd ditto - b
bored-art-40741
05/16/2016, 7:57 PM@witty-crayon-22786: Any familiarity with the 3rdparty model of node? - b
bored-art-40741
05/16/2016, 7:58 PMOur devs have been putting 3rd party deps in the package.json, which I believe is the sort of standard way to do it - b
bored-art-40741
05/16/2016, 7:59 PMI'm wondering if I need to hoist those into our 3rdparty/... target model, and if so if I'm going to need to maintain parity between those two files - b
bored-art-40741
05/16/2016, 7:59 PM(Or if there's some sort of buildgen?) - m
mysterious-action-44812
05/16/2016, 8:00 PMis there pants plugin / support for pep8? - m
mammoth-pharmacist-54215
05/16/2016, 8:00 PM@mysterious-action-44812: do you mean style checking in pants?