Skip to content

Delay release of std.range.slides until 2.076#5614

Merged
dlang-bot merged 1 commit intodlang:stablefrom
wilzbach:delay-slides
Jul 16, 2017
Merged

Delay release of std.range.slides until 2.076#5614
dlang-bot merged 1 commit intodlang:stablefrom
wilzbach:delay-slides

Conversation

@wilzbach
Copy link
Contributor

When browsing the changelog and documentation of the upcoming slides function in std.range I discovered an API issue that I never got around to fix. As the release of 2.075 is imminent, I think the most reasonable thing is to delay the release and hope we can make it in time before Martin does the final tagging.
I am really sorry for this last-minute sneak-in, but I don't want to think about doing deprecations for a new symbol ...

CC @andralex @CyberShadow @MartinNowak

(Slides PR: #4027)

@dlang-bot
Copy link
Contributor

Thanks for your pull request, @wilzbach!

Bugzilla references

Your PR doesn't reference any Bugzilla issue.

If your PR contains non-trivial changes, please reference a Bugzilla issue or create a manual changelog.

@JackStouffer
Copy link
Contributor

Can you describe the API issue?

Copy link
Member

@andralex andralex left a comment

Choose a reason for hiding this comment

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

OK, thanks for the sharp eyes.

@wilzbach
Copy link
Contributor Author

Can you describe the API issue?

withFewerElements should be the default and behave like in Scala.
Currently withFewerElements doesn't even work as expected.

I actually started a branch to fix slides a long time ago, but apparently the branch got lost and thus I forgot about it.

@dlang-bot dlang-bot merged commit 71b77d4 into dlang:stable Jul 16, 2017
@wilzbach wilzbach deleted the delay-slides branch December 11, 2017 02:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants