Skip to content

Conversation

@odersky
Copy link
Contributor

@odersky odersky commented Feb 8, 2016

Fixes #681. Review by @smarter.

Fixes $681.
@smarter
Copy link
Member

smarter commented Feb 8, 2016

It looks like this commit accidentally includes the changes of #1053
Anyway, there's 13 issues marked "Fix available": https://github.com/lampepfl/dotty/issues?q=is%3Aissue+is%3Aopen+label%3A%22Fix+available%22 For most of them, the fix has been merged and we just need to add the test case. I was planning to add all these test cases in one commit and make a PR for that, but didn't get around to it yet. Would you like me to do this today?

@odersky
Copy link
Contributor Author

odersky commented Feb 8, 2016

Yes, that would be great. Thanks! - Martin

On Mon, Feb 8, 2016 at 8:34 PM, Guillaume Martres notifications@github.com
wrote:

It looks like this commit accidentally includes the changes of #1053
#1053
Anyway, there's 13 issues marked "Fix available":
https://github.com/lampepfl/dotty/issues?q=is%3Aissue+is%3Aopen+label%3A%22Fix+available%22
For most of them, the fix has been merged and we just need to add the test
case. I was planning to add all these test cases in one commit and make a
PR for that, but didn't get around to it yet. Would you like me to do this
today?


Reply to this email directly or view it on GitHub
#1069 (comment).

Martin Odersky
EPFL

@smarter
Copy link
Member

smarter commented Feb 8, 2016

Superceded by #1071 .

@smarter smarter closed this Feb 8, 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.

2 participants