Skip to content

BREAKING CHANGE: Issue ID should be a long#2962

Merged
nickfloyd merged 3 commits intooctokit:mainfrom
kzu:issues-long
Jan 8, 2025
Merged

BREAKING CHANGE: Issue ID should be a long#2962
nickfloyd merged 3 commits intooctokit:mainfrom
kzu:issues-long

Conversation

@kzu
Copy link
Copy Markdown
Contributor

@kzu kzu commented Aug 27, 2024

According to the documentation at https://docs.github.com/en/rest/issues/issues.

Fixes #2351 for issue ids.

Does this introduce a breaking change?

Please see our docs on breaking changes to help!

  • Yes
  • No

@kfcampbell
Copy link
Copy Markdown
Contributor

This was already done in #2890. You've changed issue number, which is the "2351" in the linked issue https://github.com/octokit/octokit.net/issues/2351. The issue number increments from 1 for each new repository, and can safely be described with an integer.

@kzu
Copy link
Copy Markdown
Contributor Author

kzu commented Aug 27, 2024

image

vs

image

either one or the other is the wrong type.

@nickfloyd nickfloyd added Type: Breaking change Used to note any change that requires a major version bump Type: Maintenance Any dependency, housekeeping, and clean up Issue or PR labels Jan 8, 2025
@nickfloyd nickfloyd changed the title Issue ID should be a long BREAKING CHANGE: Issue ID should be a long Jan 8, 2025
@nickfloyd nickfloyd merged commit ffd1b40 into octokit:main Jan 8, 2025
@nickfloyd
Copy link
Copy Markdown
Contributor

Merging this in as a preventative measure since this is a manually generated SDK. These changes, while breaking should have a nominal impact on usage.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Type: Breaking change Used to note any change that requires a major version bump Type: Maintenance Any dependency, housekeeping, and clean up Issue or PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

GitHub IDs have exceeded Int32.MaxValue; Octokit uses int

3 participants