Skip to content

Conversation

@gep13
Copy link
Member

@gep13 gep13 commented Jan 9, 2025

Description

This addresses an issue which was seen when trying to close milestones
using the GitLab provider. There would be an error thrown regarding
being able to correctly serialize a number into an int32, since the
number was too big.

An upstream change was needed in NGitLab, so switch to usign to a long.
There was a similar error when using GitHub, and a bump in the version
of Octokit being used was required.

This has been tested against a GitLab repository, and the close command
which was previously failing is now working.

Related Issue

Fixes #633

Motivation and Context

An error was being thrown when attempting to use the close command when doing releases.

How Has This Been Tested?

The reproduction steps in #633 were followed both before and after this change was made.
Those steps were initially failing, and after this change was made, things were working again.

Screenshots (if appropriate):

N/A

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@gep13 gep13 requested a review from AdmiringWorm January 9, 2025 08:44
gep13 added 2 commits January 9, 2025 09:34
As answered here:

https://stackoverflow.com/a/6792308/671491

The first project in the sln file is the one that is set as the default
startup project when opened in Visual Studio. This prevents the need to
manually set this once the solution is opened.
This addresses an issue which was seen when trying to close milestones
using the GitLab provider. There would be an error thrown regarding
being able to correctly serialize a number into an int32, since the
number was too big.

An upstream change was needed in NGitLab, so switch to usign to a long.
There was a similar error when using GitHub, and a bump in the version
of Octokit being used was required.

This has been tested against a GitLab repository, and the close command
which was previously failing is now working.
@gep13 gep13 changed the title Issue 633 (#633) Update to latest NGitLab package version Jan 9, 2025
Copy link
Member

@AdmiringWorm AdmiringWorm left a comment

Choose a reason for hiding this comment

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

LGTM

@AdmiringWorm AdmiringWorm merged commit a69ce25 into GitTools:develop Jan 9, 2025
2 checks passed
@AdmiringWorm
Copy link
Member

@gep13 thanks for getting this updated. Great work 👍

@arturcic arturcic modified the milestone: 0.19.0 Jan 9, 2025
@gep13 gep13 deleted the issue-633 branch February 21, 2025 15:00
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.

The JSON value could not be converted to System.Int32

3 participants