Skip to content
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

[DOC][Overview] Use idiomatic .extract_first() #1994

Merged
merged 1 commit into from May 17, 2016
Merged

Conversation

@redapple
Copy link
Contributor

@redapple redapple commented May 17, 2016

No description provided.

@redapple redapple added the docs label May 17, 2016
@kmike
Copy link
Member

@kmike kmike commented May 17, 2016

It looks like it was me who persuaded @eliasdorneles to remove extract_first from this example: #1106 (comment). I'm fine with adding it now :)

@eliasdorneles
Copy link
Member

@eliasdorneles eliasdorneles commented May 17, 2016

Alrighty! :)

@eliasdorneles eliasdorneles merged commit ed4b9af into master May 17, 2016
@kmike kmike deleted the overview-code branch May 19, 2016
redapple added a commit that referenced this pull request Jun 8, 2016
[backport][1.1][DOC] Use idiomatic .extract_first() (PR #1994)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants