fresh-architect-74417
02/22/2021, 9:35 PMwitty-crayon-22786
02/22/2021, 9:44 PMenough-analyst-54434
02/22/2021, 9:44 PMfresh-architect-74417
02/22/2021, 9:47 PMwitty-crayon-22786
02/22/2021, 9:48 PM./pants dependencies
goal to dump out the inferred dependenciesfresh-architect-74417
02/22/2021, 9:48 PMwitty-crayon-22786
02/22/2021, 9:49 PM./pants dependencies
? yes, it isfresh-architect-74417
02/22/2021, 9:49 PMwitty-crayon-22786
02/22/2021, 9:50 PMfresh-architect-74417
02/22/2021, 9:54 PMwitty-crayon-22786
02/22/2021, 9:55 PM./pants dependencies
to do that instead. that’s backwards compatible, and supports both inference and manual dependencies.fresh-architect-74417
02/22/2021, 10:01 PM./pants dependencies
, it means I will need to run it via os.system, or a subprocess. That is extramurally slow. It means we will need to run that command on every modified target.
We had that originally implemented in first version of our continuous deployment script, and I replaced it with a custom parser. We had almost 100x improvement after I did that.witty-crayon-22786
02/22/2021, 10:02 PM./pants dependencies
once per target: that’s not what we would recommend: see https://www.pantsbuild.org/docs/using-pants-in-ci#recommended-commands instead./pants --changed-since=origin/main --changed-dependees=transitive list
in a branch, and see how long it takesfresh-architect-74417
02/22/2021, 10:03 PMwitty-crayon-22786
02/22/2021, 10:03 PM--changed-since
flag takes a git-ref as an argument.fresh-architect-74417
02/22/2021, 10:04 PMwitty-crayon-22786
02/22/2021, 10:04 PM