meta: post comment when pr labeled fast-track

[jasnell]

Automatically adds a comment when a pull requests is labeled "fast-track"

Signed-off-by: James M Snell jasnell@gmail.com

[jasnell]

@nodejs/build ... my github actions skill is still fairly weak so I'd appreciate if someone could help verify this.

[targos]

Unfortunately you cannot test changes to a workflow that uses the pull_request_target event without landing them on a branch and opening a PR against that branch

[aduh95]

Maybe this should be merged with .github/workflows/comment-stalled.yml and called something like auto-comments.yml? I suppose we may want to add more automation in the future, adding a new workflow each time seems unnecessary.

[jasnell]

@targos @aduh95 ... updated! I combined the label actions into a single workflow and I'm using the gh cli for the fast track action. The gh cli could also be used for the stalled action but I'd prefer to do that in a separate PR.

[targos]

LGTM. The file could be renamed to "comment-labeled.yml" or "labeled.yml"

[mmarchini]

Personally I would prefer the files to remain separate, merging them doesn't bring any value because there's no reuse between the two flows, and the only thing they share is the labeled event for PRs. Plus merging the files increase the complexity because now we need an extra condition in the if statement to skip the fast track job on issues.

I don't feel strong enough about it to ask to go back to separate files though.

[jasnell]

@mmarchini ... I don't know if I was doing something wrong or what, but I actually had a difficult time getting the fastTrack workflow to run consistently when it was in a separate file.... and by that I mean in testing on my personal fork, the stalled label workflow would run but the fast track wouldn't. The only way I was able to get both to run consistently was putting them both in a single workflow file. Not opposed to separating them back out into separate files if that works and is preferred but let's go ahead and land this PR and we can separate them out later if necessary in another commit.

[jasnell]

Landed in d85929c