feat: allow for releasedLabels with successComment set to false

[kristof-mattei]

As title says, this PR allow you to set successComment to false and still get the labels.

I recommend a review with whitespace changes hidden to reduce noise due to changes in indentation.

Oh, and I fixed body -> labels for posting labels: https://docs.github.com/en/rest/issues/labels?apiVersion=2022-11-28#add-labels-to-an-issue

[kristof-mattei]

@gr2m done!

[gr2m]

Hmm sorry some of the tests are failing, from the looks of it, it is most likely related to #857. Could you have a look? I'm sorry this is probably unrelated to your changes, I don't have time to look into it myself. Let us know if you don't have time either and we will get back to it another time

[kristof-mattei]

Hmm sorry some of the tests are failing, from the looks of it, it is most likely related to #857. Could you have a look? I'm sorry this is probably unrelated to your changes, I don't have time to look into it myself. Let us know if you don't have time either and we will get back to it another time

Totally, maybe not today, but definitely this week.

[kristof-mattei]

@gr2m it now works on my machine!

[babblebey]

Hi @kristof-mattei, kindly resolve the conflict 😉

[babblebey]

Hey @kristof-mattei, thank you for sticking to this.

This PR was blocked by the work going on in the PR #874, hence we haven't gotten to this yet.... the PR deprecates the ability of setting successComment to false, introducing successCommentCondition in its place.... It merges soon though 😉

But, in the mean time, kindly confirm my understanding of this feature by answering the question below...

1. Noting the introduction of the new successCommentCondition (in feat: allow conditional skip on success and fail comments #874) which accepts condition via config that results to a boolean value declaring whether both successComment and releasedLabel is added to issues/PRs.... Does this feature separate the releasedLabel from this condition??
2. If this seperate the releasedLabel from the successCommentCondition, Are we gonna be looking to introduce a similar config property releasedLabelCondition to accept dictated conditions under which a release label is added to issues/PRs??

🤔