We used to do code review on ReviewBoard. See the 1st comment by

zundel

for an explanation of how Square use{d,s} `alias`: https://rbcommons.com/s/twitter/r/3939/

Thanks

If you have time to add some more info to the docs there, it would be appreciated!

Good idea. I’ll do it later this week

see https://www.pantsbuild.org/howto_contribute.html ... the docs are generated with https://www.pantsbuild.org/docs.html

Guys I started on this today (sorry for taking the time). Just wanted to run something by you. The docs should mention both

alias

and

target

as methods for creating aliases but the latter should be used only when it’s desirable to “bundle” several targets together and perform goals on them all together. If you agree with this statement then I’ll modify the current alias docs to recommend using alias unless such a bundle is required.

that looks right!

So you’re worried that in order to alias a single project people might accidentally include other projects? The review mentioned intransitive dependencies as well in this context but I fail to understand how that’s related

Yes, correct.