Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Moving ActorRef API to scala.concurrent. #1394

Merged
merged 4 commits into from Sep 26, 2012

Conversation

Projects
None yet
4 participants
Contributor

vjovanov commented Sep 25, 2012

Moving ActorRef API to scala.concurrent.

Adding Actor Migration Kit tests.

Fixed SI-6305 which was also accidentally merged to master.

Review by @phaller

I agree that this should go into the migration package. But then we should also move the source file into the migration directory.

Here as well (see above comment)

Contributor

phaller commented Sep 25, 2012

I think after moving those files we're good.

Member

gkossakowski commented Sep 25, 2012

PLS REBUILD ALL.

Contributor

vjovanov commented Sep 25, 2012

Fixed the reveiw comments.

Contributor

phaller commented Sep 25, 2012

LGTM

Member

gkossakowski commented Sep 25, 2012

PLS REBUILD pr-rangepos

Member

gkossakowski commented Sep 26, 2012

PLS REBUILD ALL

gkossakowski added a commit that referenced this pull request Sep 26, 2012

Merge pull request #1394 from vjovanov/amk/ActorRef
Moving ActorRef API to scala.concurrent.

@gkossakowski gkossakowski merged commit 5691373 into scala:2.10.x Sep 26, 2012

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