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 SelectProjection #5672

Merged
merged 1 commit into from Apr 6, 2018

Conversation

Projects
None yet
3 participants
@stuhood
Copy link
Member

stuhood commented Apr 6, 2018

SelectProjection is entirely superseded by the new @rule coroutine syntax from #5580.

@stuhood stuhood requested review from baroquebobcat , illicitonion and ity Apr 6, 2018

@kwlzn

kwlzn approved these changes Apr 6, 2018

Copy link
Member

kwlzn left a comment

lgtm

@stuhood stuhood merged commit 79d9d35 into pantsbuild:master Apr 6, 2018

1 check failed

continuous-integration/travis-ci/pr The Travis CI build failed
Details

@stuhood stuhood deleted the twitter:stuhood/remove-select-projection branch Apr 6, 2018

@baroquebobcat

This comment has been minimized.

Copy link
Contributor

baroquebobcat commented Apr 6, 2018

Nice!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment