Skip to content

Merging in the big extra context pull request#274

Merged
pydanny merged 11 commits intomasterfrom
audreyr-extra-context
Sep 2, 2014
Merged

Merging in the big extra context pull request#274
pydanny merged 11 commits intomasterfrom
audreyr-extra-context

Conversation

@audreyfeldroy
Copy link
Copy Markdown
Member

This pull request is my merging-in of #260. It was quite complicated (see my comments in #260 for details), so the purpose of this PR is just to make it clear what exactly the changes are.

Since it was complicated and delicate, can another core committer please give this a quick review?

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.02%) when pulling 2a1f47e on audreyr-extra-context into a944be3 on master.

@audreyfeldroy
Copy link
Copy Markdown
Member Author

A couple notes about this:

pydanny added a commit that referenced this pull request Sep 2, 2014
Merging this in, all tests pass. Tomorrow evening I hope to take a stab at covering tests for #273 and #275. As well as working on additional documentation and example usages of this feature.
@pydanny pydanny merged commit d0fa8b4 into master Sep 2, 2014
@hackebrot hackebrot deleted the audreyr-extra-context branch February 11, 2016 01:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants