Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good call.
@@ -21,14 +21,15 @@ pythonpath: [ | |||
] | |||
backend_packages: +[ | |||
'twitter.common.pants.python.commons', | |||
'pants.contrib.python.checks', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
intentional?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes: Putting something in the plugin list enables it, so having this here was enabling it twice, which now causes an error.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ah, cool
As evidenced by one shard failing with a timeout at the end of pytest running, and the other not, it looks like this has nothing to do with the daemon. Regardless, will skip enabling it in this bump because I've already made enough edits. |
No description provided.