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

Remove pool whitelist #7633

Merged
merged 1 commit into from Sep 13, 2018

Conversation

Projects
None yet
2 participants
@naderman
Member

naderman commented Sep 12, 2018

Part of #7630

@naderman naderman added this to the 2.0 milestone Sep 12, 2018

@naderman naderman changed the base branch from master to 2.0 Sep 12, 2018

@stof

This comment has been minimized.

Show comment
Hide comment
@stof

stof Sep 12, 2018

Contributor

shouldn't we have code added in PoolBuilder too ?

Contributor

stof commented Sep 12, 2018

shouldn't we have code added in PoolBuilder too ?

@naderman

This comment has been minimized.

Show comment
Hide comment
@naderman

naderman Sep 13, 2018

Member

@stof no the pool builder already does this, it was just redundant at this point

Member

naderman commented Sep 13, 2018

@stof no the pool builder already does this, it was just redundant at this point

@naderman naderman merged commit 53c5e3c into composer:2.0 Sep 13, 2018

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment