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

Add an example on pool usage in the documentation #1217

Merged
merged 1 commit into from
Mar 28, 2016

Conversation

ledsusop
Copy link

initial pull request and doc change

@landscape-bot
Copy link

Code Health
Code quality remained the same when pulling 3107b39 on ledsusop:master into 7e50742 on airbnb:master.

@bolkedebruin
Copy link
Contributor

Hi lusop. Thanks for the contribution! Please follow these http://chris.beams.io/posts/git-commit/ guidelines for your commit message. It helps creating changelogs and release notes in the future. Thanks!

@ledsusop
Copy link
Author

alright @bolkedebruin , do you need me to reissue a pull request?

@bolkedebruin
Copy link
Contributor

Just update your commit message. Git commit --amend , git push --force

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.1%) to 66.27% when pulling 3107b3967c8f5678dda61a6ec689caa04549b462 on ledsusop:master into 7e50742 on airbnb:master.

@ledsusop ledsusop changed the title added pools example in the documentation of concepts as suggested by sid Add an example on pool usage in the documentation Mar 26, 2016
@ledsusop
Copy link
Author

done

@landscape-bot
Copy link

Code Health
Repository health increased by 0.02% when pulling 2f48a53 on ledsusop:master into 7a632e2 on airbnb:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 66.377% when pulling 2f48a53 on ledsusop:master into 7a632e2 on airbnb:master.

@bolkedebruin bolkedebruin merged commit 1db892b into apache:master Mar 28, 2016
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