Skip to content

September 2018 SIP minutes #1171

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

Merged
merged 9 commits into from
Oct 18, 2018
Merged

September 2018 SIP minutes #1171

merged 9 commits into from
Oct 18, 2018

Conversation

darjutak
Copy link
Contributor

@sjrd and @jvican please take a look, August one should follow

@milessabin
Copy link
Contributor

Small nit ... I wasn't present at this meeting.

Miles moved, not present
@darjutak
Copy link
Contributor Author

Thank you Miles, just changed it

Copy link
Member

@sjrd sjrd left a comment

Choose a reason for hiding this comment

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

Thank you. I just have a few comments.


#### [“Proposal to remove early initializers from the language”](https://contributors.scala-lang.org/t/proposal-to-remove-early-initializers-from-the-language/2144)

([YouTube time: 19’29’ - 20’40’’](https://youtu.be/tEb4UF6RJrM?t=1169))
Copy link
Member

Choose a reason for hiding this comment

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

Is it normal that there is nothing but the Youtube link in this section?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@jvican can you please write up a line here, it's like a minute that is missing


An overview of the second batch can be found [in this Scala Contributors thread](https://contributors.scala-lang.org/t/second-batch-of-scala-3-sips-additions-to-scalas-type-system/2376). The batches under discussion are:

1. https://contributors.scala-lang.org/t/proposal-to-add-trait-parameters-to-the-language/2356/11
Copy link
Member

Choose a reason for hiding this comment

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

That URL points to the 11th post in the thread. You should remove the trailing /11 to point the thread start.

1. https://contributors.scala-lang.org/t/proposal-to-add-trait-parameters-to-the-language/2356/11
2. https://contributors.scala-lang.org/t/proposal-to-add-intersection-types-to-the-language/2351/1
3. https://contributors.scala-lang.org/t/proposal-to-add-union-types-to-the-language/2352/1
4. https://contributors.scala-lang.org/t/proposal-to-add-dependent-function-types-to-the-language/2354/1
Copy link
Member

Choose a reason for hiding this comment

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

Ideally, also remove the trailing /1 on these three URLs.

([YouTube time: 20’49’’ - 24'01](https://youtu.be/tEb4UF6RJrM?t=1250))

**Martin** presents the intersection types as per doc. He does a basic
description of the feature and points ou8t that intersection types are the
Copy link
Member

Choose a reason for hiding this comment

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

ou8t -> out

([YouTube time: 44’42’’ - end ](https://youtu.be/tEb4UF6RJrM?t=2682))

**Martin** describes trait parameters and says that they subsume a large number of use
cases of early initializers. They were not added to Scala 2 because of
Copy link
Member

Choose a reason for hiding this comment

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

"to Scala 2" is misleading here I think. I think a better rendering of the idea is "to Scala from the start". We might still add them to Scala 2 (hopefully we will). The point Martin made was that in initially designing Scala, the only reason he did not include trait parameters was the uncertainty [...].

@darjutak
Copy link
Contributor Author

@sjrd thanks for your eagle-eye review, on it!

@darjutak darjutak merged commit 848d693 into scala:master Oct 18, 2018
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.

3 participants