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

Added method to flatten paged content into one async collection #744

Merged

Conversation

ObsidianMinor
Copy link
Contributor

Now you can use LINQ with your async collections. Have fun.

… method. Also fixed ClientHelper using incorrect guild batch count.
@ObsidianMinor
Copy link
Contributor Author

This could wait until C# 8 so I can replace the PagedCollectionEnumerator with just a yield return

@Still34
Copy link
Member

Still34 commented Sep 11, 2017

Wouldn't the release take another year or so?

@ObsidianMinor
Copy link
Contributor Author

Discord.Net 2.x could also take another year. So it depends on if both C# 8 and the next major version are released near each other I guess.

@Still34
Copy link
Member

Still34 commented Sep 11, 2017

I don't think 2.0 will take that long to release, as 2.0 includes some breaking changes that need to be pushed ASAP. 3.0 would make more sense for integrating C#8 changes.

@foxbot foxbot merged commit 5bbd9bb into discord-net:dev Jan 7, 2018
FiniteReality pushed a commit to FiniteReality/Discord.Net that referenced this pull request May 5, 2018
… method. Also fixed ClientHelper using incorrect guild batch count. (discord-net#744)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Library Development
Future: Breaking (2.x)
Development

Successfully merging this pull request may close these issues.

None yet

3 participants