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

Update project #13

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Update project #13

wants to merge 2 commits into from

Conversation

laughedelic
Copy link
Contributor

No description provided.

@laughedelic laughedelic self-assigned this Oct 17, 2017
@laughedelic
Copy link
Contributor Author

laughedelic commented Oct 23, 2017

This doesn't compile with updated dependencies. Something caused by changes in AnyDataSet -> AnyRecord.of[AnyData] -> AnyProduct...

type Output <: AnyDataSet
// FIXME: this is supposed to be the same:
// type Output <: AnyDataSet // = AnyRecordType.Of[AnyData]
type Output <: AnyRecordType { type Keys <: AnyProductType { type Types <: AnyKList { type Bound <: AnyData } } }
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't understand why it doesn't compile with AnyDataSet, which is AnyRecordType.Of[AnyData], which is supposed to be the same as the last line (which makes it compile):

case object AnyRecordType {
  type Of[+B <: AnyType] = AnyRecordType { type Keys <: AnyProductType.Of[B] }
}
case object AnyProductType {
  type Of[+B <: AnyType] = AnyProductType { type Types <: AnyKList.Of[B] }
}
case object AnyKList {
  type Of[+B] = AnyKList { type Bound <: B }
}

@eparejatobes any clue?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@laughedelic
Copy link
Contributor Author

laughedelic commented Oct 23, 2017

OK, So I'm going to leave here this unfolded refined type and hope that it doesn't break anything dependent on it. I think these Tasks are not used anywhere anyway

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

Successfully merging this pull request may close these issues.

2 participants