#general
- b
bored-art-40741
02/01/2016, 11:10 PMI'm also fine with being explicit about things being public, however I do not think we can get away with not being explicit about things being private - b
bored-art-40741
02/01/2016, 11:10 PMIt needs to be visually obvious when importing that you are not using a public API - b
bored-art-40741
02/01/2016, 11:11 PMFor this reason I think the only viable solution involves module naming conventions that somehow scream "private!" at the user - b
bored-art-40741
02/01/2016, 11:11 PMRegardless of what the details of that naming convention are - r
rough-minister-58256
02/01/2016, 11:12 PMI worry that we’re making sweeping changes like this that add overhead at the expense of pants developers for the handful of ppl who actually use private apis out there. - w
witty-crayon-22786
02/01/2016, 11:12 PMi disagree with the "visually obvious" requirement. that's a nice to have. - r
rough-minister-58256
02/01/2016, 11:12 PMI almost htink we could get away wityh just a clear doc - r
rough-minister-58256
02/01/2016, 11:12 PMsame - m
mammoth-pharmacist-54215
02/01/2016, 11:12 PM@rough-minister-58256: do you mean a list of public api’s or something similar? - r
rough-minister-58256
02/01/2016, 11:12 PMyeah - r
rough-minister-58256
02/01/2016, 11:13 PM‘pants public api' - r
rough-minister-58256
02/01/2016, 11:13 PMor just simple annotations in whatever apidoc we eventually gen - r
rough-minister-58256
02/01/2016, 11:13 PMwith a clear policy - r
rough-minister-58256
02/01/2016, 11:13 PMin the dev guide - w
witty-crayon-22786
02/01/2016, 11:13 PMwe want more private apis, and if 95% of apis are private and have _ in them, it's a massive amount of noise. - b
bored-art-40741
02/01/2016, 11:13 PMHeh, let me be clear, my position here is that we shouldn't do any of this, that we should just document APIs as being public or private, and that we should spend all of this effort writing tests to enforce these requirements - r
rough-minister-58256
02/01/2016, 11:13 PMagree^ - w
witty-crayon-22786
02/01/2016, 11:13 PM+1 - w
witty-crayon-22786
02/01/2016, 11:13 PMit's settled then. docstrings =P - r
rough-minister-58256
02/01/2016, 11:14 PMya - m
mammoth-pharmacist-54215
02/01/2016, 11:14 PM+1 - b
bored-art-40741
02/01/2016, 11:14 PMWell, I think Benjy and Eric should be involved in a re-evaluation of this decision - w
witty-crayon-22786
02/01/2016, 11:14 PMagreed. - b
bored-art-40741
02/01/2016, 11:14 PMSince they spearheaded this in the first place - m
mammoth-pharmacist-54215
02/01/2016, 11:14 PMagree - r
rough-minister-58256
02/01/2016, 11:14 PMgood summit material. - b
bored-art-40741
02/01/2016, 11:14 PMyeah good point - r
rough-minister-58256
02/01/2016, 11:14 PMsince Eric is out for the next 3 weeks - b
bored-art-40741
02/01/2016, 11:14 PMWe should start a doc if we haven't already - m
mammoth-pharmacist-54215
02/01/2016, 11:14 PMthere is a doc already started