Jericho
Jericho
> Related to the above, if I use this: > > > I can get the associated MR for the issue in question, so that makes me wonder if I...
Changes since last review: - Added a configuration option so users can specify whether they want contributors to be included in the release notes - Created a separate template which...
Upgrading Octokit is a good first step but it's not sufficient to resolve this issue. I was able to reproduce this problem even after upgrading the reference. Here's a more...
After further investigation it appears that Octokit still defines `Release.Id` and `ReleaseAsset.Id` as `int` therefore we shouldn't change these two properties in our model classes.
I can confirm that after upgrading the Octokit reference and changing `IssueComment.Id` to `long`, my build conpletes successfully ``` ======================================== Publish-GitHub-Release ======================================== ____ _ _ ____ _ __ __ /...
Submitted a PR with necessary changes. I did not change Milestone.InternalNumber to long since it's not needed to resolve this issue but I'll be happy to amend my PR if...
I would be happy to contribute. I will start with combing through the code base to identify the various classes that would be affected.
> I think the easiest way to get this done is to fix them one by one instead of batches. @elachlan Thanks for the guidance. I'll also keep @RussKie 's...
Here's the list of classes that will benefit from replacing `ArrayList` with `List` (I'll continuously maintain this list as I discover more classes): - [x] WeakRefCollection: use `List` instead of...