New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

execution_strategy is a memoized_property #6052

Merged
merged 1 commit into from Jul 10, 2018

Conversation

Projects
None yet
2 participants
@illicitonion
Copy link
Contributor

illicitonion commented Jun 29, 2018

Right now, there are two ways of looking it up, and one bypasses caching.

@illicitonion illicitonion requested a review from dotordogh Jun 29, 2018

@dotordogh
Copy link
Contributor

dotordogh left a comment

Looks great! Thank you!!

@illicitonion illicitonion force-pushed the twitter:dwagnerhall/memoize branch from 7ee0f5b to b58f286 Jul 10, 2018

execution_strategy is a memoized_property
Right now, there are two ways of looking it up, and one bypasses caching.

@illicitonion illicitonion force-pushed the twitter:dwagnerhall/memoize branch from b58f286 to 2465422 Jul 10, 2018

@illicitonion illicitonion merged commit 875b0cf into pantsbuild:master Jul 10, 2018

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@illicitonion illicitonion deleted the twitter:dwagnerhall/memoize branch Jul 10, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment