Skip to content

Update status of proposals #637

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 6 commits into from
Jan 10, 2017

Conversation

jvican
Copy link
Member

@jvican jvican commented Dec 2, 2016

This PR goes through outdated proposals and update them with their latest status. Review by @SethTisue | @martijnhoekstra.

@@ -3,7 +3,7 @@ layout: sip
title: SIP-18 - Modularizing Language Features

vote-status: accepted
vote-text: This is in Accept status, pending implementation. Let the record show Paul is against it.
vote-text: This SIP has already been accepted with the vote against of Paul.
Copy link
Member

Choose a reason for hiding this comment

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

I prefer the original wording. Why did this need to be updated?

Copy link
Member Author

Choose a reason for hiding this comment

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

I just wanted to emphasize that it was accepted, meaning that it's been already merged into scalac. Will change.

@@ -4,7 +4,7 @@ disqus: true
title: SIP-20 - Improved Lazy Vals Initialization

vote-status: under review
vote-text: Next iteration takes place in October 2016.
vote-text: Dormant. This proposal lacks an implementation for Scalac and is looking for a new author.
Copy link
Member

Choose a reason for hiding this comment

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

Looking for a new owner, not author.

Copy link
Member Author

Choose a reason for hiding this comment

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

👍

@@ -4,7 +4,7 @@ disqus: true
title: SIP-22 - Async

vote-status: postponed
vote-text: Authors have asked to postpone until further notice. They will eventually open the proposal when some details in the implementation are ready. Check <a href="http://localhost:4000/sips/minutes/sip-minutes.html">July's minutes</a>.
vote-text: Authors have asked to postpone this proposal until further notice. They will eventually open the proposal when some details in the implementation are ready. Check <a href="http://localhost:4000/sips/minutes/sip-minutes.html">July's minutes</a>.
Copy link
Member

Choose a reason for hiding this comment

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

Isn't this dormant? They don't plan on picking it up.

Copy link
Member Author

Choose a reason for hiding this comment

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

Yeah, dormant is more appropriate in this case.

@@ -2,6 +2,9 @@
layout: sip
disqus: true
title: SIP-23 - Literal-based singleton types

vote-status: dormant
vote-text: This proposal is dormant because original authors are not available for its review. There is an implementation <a href="https://github.com/scala/scala/pull/5310">here</a>.
Copy link
Member

Choose a reason for hiding this comment

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

Did you test this locally? I was under the impression that links don't work in front-matter.

Copy link
Member

Choose a reason for hiding this comment

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

Again, owner, not author.

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, they work.

vote-status: under review
vote-text: The following proposal needs to be updated, since only the specialized case version (with new lines) has been accepted. For more information, check the <a href="http://docs.scala-lang.org/sips/minutes/sip-20th-september-minutes.html">minutes</a>.
vote-status: approved
vote-text: The following proposal has been approved but the spec needs to be updated and the implementation has to be ported from Parsers to Scanner. For more information, check the minutes in November 2017.
Copy link
Member

Choose a reason for hiding this comment

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

November 2017? Do you mean November 2016? In any case, these minutes do not exist yet, so it's impossible to check this. Perhaps omit this until you have done the minutes?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, I will omit it for now. Thanks for the catch in the date.

@@ -2,6 +2,9 @@
layout: sip
title: SIP 25 - Trait Parameters
disqus: true

vote-status: under review
vote-text: The board agreed to schedule the next iteration of the evaluation process in 6 months, since there’s no implementation yet and the authors need time to produce one.
Copy link
Member

Choose a reason for hiding this comment

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

I was under the impression that this was dormant and needs a new owner. Martin was not going to implement this.

Copy link
Member Author

Choose a reason for hiding this comment

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

This is extracted from the Outcome section of July's minutes. In that meeting, we agreed that we would give authors 6 months for a second review. If there is no implementation in that review, we can put it as dormant, but it's not yet.

@@ -2,6 +2,9 @@
layout: sip
title: SIP-24 - Repeated By Name Parameters
disqus: true

vote-status: dormant
vote-text: Looking for a new author. This proposal needs to be updated according to the <a href="https://www.youtube.com/watch?v=hrxp4PGbi6s">SIP meeting in November 2016</a>.
Copy link
Member

Choose a reason for hiding this comment

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

Owner, not author.

@SethTisue
Copy link
Member

SethTisue commented Jan 9, 2017

@jvican hoping we could get this across the finish line...?

@jvican
Copy link
Member Author

jvican commented Jan 9, 2017

Hey @SethTisue, updating tomorrow! Sorry this fell through the cracks.

@jvican
Copy link
Member Author

jvican commented Jan 10, 2017

@heathermiller This is ready for review again 😄

@heathermiller
Copy link
Member

Thanks @jvican and @SethTisue!! ❤️

@heathermiller heathermiller merged commit 9118ecb into scala:master Jan 10, 2017
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