https://pantsbuild.org/ logo
#development
Title
# development
h

hundreds-breakfast-49010

02/11/2020, 8:02 AM
any chance I could get a few more eyes from non-toolchain people on https://github.com/pantsbuild/pants/pull/9077 ?
w

witty-crayon-22786

02/11/2020, 3:57 PM
could you summarize your reaction to https://pantsbuild.slack.com/archives/C0D7TNJHL/p1580940954159200 on there?
h

hundreds-father-404

02/11/2020, 4:19 PM
One thing that isn’t mentioned: the possibility of
./pants repl --only=ipython
. It’d be good to document that it’s a possibility. Also, to reiterate, the reason I do not like
./pants repl --only
is that the option must be provided for things to work, unlike any other goals we have in Pants (outside of
./pants login --to
). That’s a smell to me that this should instead be several dedicated goals. It’s more discoverable to find several distinct goals via
./pants goals
than it is to first run
./pants repl
, have it fail, realize you need to use
./pants repl --only
, and have to frequently check
./pants help repl
to remember what those options are.
w

witty-crayon-22786

02/11/2020, 5:06 PM
there are lessons to be learned from v1 in the regard... currently, there is a default repl per language, and you can override it. would be good to think about how that looks here.
will review in a bit.
...sorry, have been slammed. may not get to this until tomorrow.