Handle event ids greater than int32.maxvalue

[matt-richardson]

Fixes #1939

[hnrkndrssn]

Looks good to me 👍

[ryangribble]

Thanks @matt-richardson

[matt-richardson]

Thanks for the quick response. Whats your thoughts around when this could make it into a release?

[ryangribble]

I've been meaning to push a release already however there wasn't anything that was urgent/pressing. This PR changes that though so I'll try to do it in the next few days.

In the meantime you can pull the package built from merging this PR from our appveyor nuget feed if you need to get back up and running 👍

[ryangribble]

release_notes: Update EventInfo.Id field from int to long to prevent overflow exceptions

[adamralph]

Any update on getting this released? We're currently blocked on this over at https://github.com/Particular

[hnrkndrssn]

@adamralph you could use the beta package https://github.com/octokit/octokit.net/blob/master/README.md#beta-packages

That's how we unblocked ourselves while waiting for the next release.

But that might not be an option or the ideal solution...

[adamralph]

@hnrkndrssn in our case, we're using LINQPad scripts, and unfortunately we can't bake a package source into the script definitions.

[Eilon]

I just hit this as well with some events with IDs greater than int.MaxValue. Would love to see a public release that has this fix!

[Thraka]

I just ran into this issue too.

var results = await client.Issue.Events.GetAllForRepository("dotnet", "docs");

Using octokit 0.36.0

[shiftkey]

@Thraka please open a new issue as I suspect that requires a different fix to what was addressed in this PR.