Breaking issue. Bypass Rules not working anymore after update 2.5.801 about workitemupdater

[Inlustris]

@BlueBasher Is it possible to look at this ASAP? Would like to use it but now i have to bypass the step because it fails.

[Rachel-Brickey]

EDIT: I have created a PR for this fix. #94

We recently had a similar problem where the WorkItemUpdater would say it updated the status of several tickets but did not save any changes to status.

@Inlustris
The updater will work as intended if you de-select the "bypass rules" checkbox. The "bypass rules" checkbox is no longer properly wired up to do what it is meant to do. I had to clone the WorkItemUpdater repo and compile it to find the problem (see below).

@BlueBasher
On line 366 of WorkItemUpdater.ts, settings.bypassRules is passed into workItemTrackingClient.updateWorkItem(...) as the 5th parameter when it really should be the 6th. validateOnly is the 5th parameter, which tells the API not to save any changes from the request, so no updates are applied when the "bypass rules" checkbox is selected.

WorkItemUpdater/src/WorkItemUpdater/WorkItemUpdater.ts

Line 366 in 9275921

settings.bypassRules);

From src\WorkItemUpdater\node_modules\azure-devops-node-api\WorkItemTrackingApi.d.ts:

[BlueBasher]

A new version has now been released with this fix, had some issues with the release pipelines... Thanks for the help with this @rbrickey-rogue