Skip to content

Less _push and _pull#431

Merged
kwmsmith merged 4 commits into
enthought:masterfrom
cowlicks:less-push-pull-nonsense
Jun 11, 2014
Merged

Less _push and _pull#431
kwmsmith merged 4 commits into
enthought:masterfrom
cowlicks:less-push-pull-nonsense

Conversation

@cowlicks
Copy link
Copy Markdown
Contributor

Replace the push, execute, pull idiom in a few places.

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage decreased (-0.2%) when pulling 4de6211 on cowlicks:less-push-pull-nonsense into 084e5b3 on enthought:master.

@kwmsmith kwmsmith added this to the 0.4 milestone Jun 11, 2014
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe this method is refactored in another PR to remove all communication, correct?

@kwmsmith
Copy link
Copy Markdown
Contributor

👍

kwmsmith added a commit that referenced this pull request Jun 11, 2014
@kwmsmith kwmsmith merged commit e4acdac into enthought:master Jun 11, 2014
@cowlicks cowlicks deleted the less-push-pull-nonsense branch June 11, 2014 19:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants