https://pantsbuild.org/ logo
Powered by
#general
  • e

    enough-analyst-54434

    07/29/2015, 4:50 PM
  • m

    mammoth-pharmacist-54215

    07/29/2015, 4:51 PM
    I’ll look into having this behavior in the task so we are consistent.
  • m

    mammoth-pharmacist-54215

    07/29/2015, 4:51 PM
    Thanks for the feedback John
  • e

    enough-analyst-54434

    07/29/2015, 4:51 PM
    yw
  • m

    mammoth-pharmacist-54215

    07/29/2015, 4:51 PM
    do you have any opinion on separate tasks vs plugins?
  • e

    enough-analyst-54434

    07/29/2015, 4:51 PM
    @fast-megabyte-33579: does register.py seem crazy or is that just news to you?
  • e

    enough-analyst-54434

    07/29/2015, 4:52 PM
    It would be great to have feedback on the extension / customization mechanisms
  • w

    witty-crayon-22786

    07/29/2015, 4:52 PM
    @enough-analyst-54434: re: "they always run": both scalastyle and checkstyle have a 
    --skip
    flag though
  • e

    enough-analyst-54434

    07/29/2015, 4:52 PM
    ...by default
  • e

    enough-analyst-54434

    07/29/2015, 4:52 PM
    You're correct
  • f

    fast-megabyte-33579

    07/29/2015, 4:53 PM
    john.sirois: it was news to me, and I've written my own pants plugin (to do wiki publishing)
  • e

    enough-analyst-54434

    07/29/2015, 4:53 PM
    I guess the main point is it would be nice for them to all work the same however that is unless somehow the individual language conventions just don't jive in some strong way
  • f

    fast-megabyte-33579

    07/29/2015, 4:53 PM
    I always thought I was missing how checkstyle worked simple smile
  • e

    enough-analyst-54434

    07/29/2015, 4:53 PM
    Aha
  • w

    witty-crayon-22786

    07/29/2015, 4:53 PM
    @fast-megabyte-33579: to be clear, the checkers for python mostly don't exist yet
  • w

    witty-crayon-22786

    07/29/2015, 4:53 PM
    ...in pants
  • w

    witty-crayon-22786

    07/29/2015, 4:54 PM
    ...with the exception of python-eval (which is quite cool, and which twitter isn't using yet either)
  • f

    fast-megabyte-33579

    07/29/2015, 4:54 PM
    that's fine, @witty-crayon-22786 - we're mostly using scala + java in our pants repo
  • w

    witty-crayon-22786

    07/29/2015, 4:54 PM
    ah
  • f

    fast-megabyte-33579

    07/29/2015, 4:55 PM
    currently we run linters as a pre-code-review stage
  • e

    enough-analyst-54434

    07/29/2015, 4:55 PM
    I did just search through the docsite and, although mentioned, customizing your pants install for a repo is no-where treated 1st class - I just found my doc fix-it week project
  • w

    witty-crayon-22786

    07/29/2015, 4:56 PM
    to make this a bit easier, could install all of these tasks by default in their respective backends, but make them default to 
    --skip=True
  • f

    fast-megabyte-33579

    07/29/2015, 4:56 PM
    john.sirois: it is often difficult to see which parts are needed to develop pants and which parts are "things you probably want in your repo, but you need to copy some files"
  • e

    enough-analyst-54434

    07/29/2015, 4:57 PM
    @fast-megabyte-33579: agreed
  • e

    enough-analyst-54434

    07/29/2015, 4:57 PM
    @witty-crayon-22786: maybe agreed - definitely agreed with an engine that doesn't schedule tasks with no work to do
  • w

    witty-crayon-22786

    07/29/2015, 5:10 PM
    +1
  • w

    witty-crayon-22786

    07/29/2015, 5:11 PM
    how difficult would it be to extract a 
    select[_targets]
    method from 
    execute
    ? can we already envision what that will look like in the new engine
  • w

    witty-crayon-22786

    07/29/2015, 5:11 PM
    ?
  • w

    witty-crayon-22786

    07/29/2015, 5:12 PM
    (would love to be able to start incremental work in that direction now.)
  • e

    enough-analyst-54434

    07/29/2015, 5:14 PM
    I'd much rather have folks working on the engine. At the summit neither Foursquare nor Twitter had bandwidth so I said I'd sketch out a system off-to-the-side running unit tests against task scenarios ... do you have bandwidth now?
1...868788...2420Latest