You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
var issueUpdate = new IssueUpdate{Body = description};
await client.Issue.Update(owner, name, issueNumber, issueUpdate);
Github deletes some elements of the issue, this is a known scenario for us, let's create an issue so this doesn't happen again, I think there are two instance in which we update the issue so just test that when we are updating an issue we always share all the elements of an issue not only the part we want to update.
Release Note Category
Feature changes/additions
Bug fixes
Internal Infrastructure Improvements
Release Note Description
No release note needed
The text was updated successfully, but these errors were encountered:
It goes through the whole process of creating the issue and deleting it, even though we are just testing the update. I think this is okay, but I also wonder if just updating a previously opened issue would be enough.
The problem with the first option is that we are performing more operations that could fail and are not tested, but it should be fine. Additionally, we are creating an issue every time, so we would end up with a bunch of issues (all of them closed, of course).
The problem with the second option is that we are not controlling the whole scenario and we are not starting with a clean state every time. Some people can manually remove the milestone and such.
I went for the first option of having a clean state every time, but if anyone has concerns about this, please let me know.
When using something like the following
Github deletes some elements of the issue, this is a known scenario for us, let's create an issue so this doesn't happen again, I think there are two instance in which we update the issue so just test that when we are updating an issue we always share all the elements of an issue not only the part we want to update.
Release Note Category
Release Note Description
No release note needed
The text was updated successfully, but these errors were encountered: