Un-enforcing status check doesn't work as expected

[alexandresavicki]

Hi @SvanBoxel great job on this APP.

I found an possible bug when i configure enforce: true and enforce_admin: true, at this point everything works like a charm.
But when i try to set enforce: false and enforce_admin: false after the workflow already created checks on protection branchs it keeps as true for all repositoryes in my organization.

It would be great if, after the change forces it to false, it is updated checks for all repositoires.

[SvanBoxel]

Bug confirmed. 👍🏽

At the moment the app copies all current required status checks as an array and then, depending on whether you enforce it or not, adds a new required check to that array. That means that even though you toggle the boolean from true to false, it will still be part of the original array of required checks.

Culprit:

organization-workflows/src/utils/enforce-protection.ts

Line 39 in 0227bef

contexts: [...(protection ? protection.data.required_status_checks.contexts : []), context_name]

[SvanBoxel]

Release with 1.3.0.